You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@lens.apache.org by Akshay Goyal <ak...@gmail.com> on 2015/08/27 11:49:10 UTC

Re: Review Request 37716: LENS-753: Queue number for queries in submitted queue

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37716/
-----------------------------------------------------------

(Updated Aug. 27, 2015, 9:49 a.m.)


Review request for lens and Amareshwari Sriramadasu.


Bugs: LENS-753
    https://issues.apache.org/jira/browse/LENS-753


Repository: lens


Description
-------

Queue number for queries in submitted queue


Diffs
-----

  lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java 9614caa 
  lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java 7a5b177 
  lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryStatus.java f78b7c3 
  lens-server-api/src/main/java/org/apache/lens/server/api/query/FinishedLensQuery.java 6cecf7e 
  lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java beaa72f 
  lens-server-api/src/main/java/org/apache/lens/server/api/query/collect/ImmutableQueryCollection.java 344ff10 
  lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java c29a1ac 
  lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultEstimatedQueryCollection.java e3505bb 
  lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultQueryCollection.java f9e7701 
  lens-server/src/main/java/org/apache/lens/server/query/collect/MutableQueryCollection.java e6e777c 
  lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeEstimatedQueryCollection.java cdbd2ad 
  lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeQueryCollection.java 7b43a38 
  lens-server/src/test/java/org/apache/lens/server/query/collect/DefaultQueryCollectionTest.java 7a81e83 
  lens-server/src/test/java/org/apache/lens/server/query/collect/QueryCollectUtil.java 51fcf00 
  src/site/apt/user/cli.apt be93b8c 

Diff: https://reviews.apache.org/r/37716/diff/


Testing
-------

[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO] 
[INFO] Lens Checkstyle Rules ............................. SUCCESS [1.847s]
[INFO] Lens .............................................. SUCCESS [2.853s]
[INFO] Lens API .......................................... SUCCESS [24.281s]
[INFO] Lens API for server and extensions ................ SUCCESS [21.248s]
[INFO] Lens Cube ......................................... SUCCESS [5:16.136s]
[INFO] Lens DB storage ................................... SUCCESS [19.132s]
[INFO] Lens Query Library ................................ SUCCESS [14.881s]
[INFO] Lens Hive Driver .................................. SUCCESS [2:49.676s]
[INFO] Lens Driver for JDBC .............................. SUCCESS [38.935s]
[INFO] Lens Elastic Search Driver ........................ SUCCESS [16.642s]
[INFO] Lens Server ....................................... SUCCESS [5:35.820s]
[INFO] Lens client ....................................... SUCCESS [36.048s]
[INFO] Lens CLI .......................................... SUCCESS [2:35.600s]
[INFO] Lens Examples ..................................... SUCCESS [8.941s]
[INFO] Lens Distribution ................................. SUCCESS [8.296s]
[INFO] Lens ML Lib ....................................... SUCCESS [1:21.192s]
[INFO] Lens ML Ext Distribution .......................... SUCCESS [2.212s]
[INFO] Lens Regression ................................... SUCCESS [11.972s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 21:06.610s
[INFO] Finished at: Mon Aug 24 15:26:07 UTC 2015
[INFO] Final Memory: 193M/1887M
[INFO] ------------------------------------------------------------------------


Thanks,

Akshay Goyal

Re: Review Request 37716: LENS-753: Queue number for queries in submitted queue

Posted by Akshay Goyal <ak...@gmail.com>.


> On Aug. 27, 2015, 10:03 a.m., Rajat Khandelwal wrote:
> > lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java, lines 126-132
> > <https://reviews.apache.org/r/37716/diff/1/?file=1048240#file1048240line126>
> >
> >     If queueNumber is not always populated, we should use `Integer` instead of `int`.

This will always be set.


> On Aug. 27, 2015, 10:03 a.m., Rajat Khandelwal wrote:
> > lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java, lines 215-219
> > <https://reviews.apache.org/r/37716/diff/1/?file=1048241#file1048241line215>
> >
> >     This code can be refactored out, it's common in this function and the one above it.
> >     
> >     Though, I think let's not add the two new functions. `query status`, `query queueNumber`, `query progress` all are fetching the full status.

Makes sense. Removed both of them.


- Akshay


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37716/#review96664
-----------------------------------------------------------


On Aug. 27, 2015, 1:34 p.m., Akshay Goyal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37716/
> -----------------------------------------------------------
> 
> (Updated Aug. 27, 2015, 1:34 p.m.)
> 
> 
> Review request for lens and Amareshwari Sriramadasu.
> 
> 
> Bugs: LENS-753
>     https://issues.apache.org/jira/browse/LENS-753
> 
> 
> Repository: lens
> 
> 
> Description
> -------
> 
> Queue number for queries in submitted queue
> 
> 
> Diffs
> -----
> 
>   lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java 9614caa 
>   lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java 7a5b177 
>   lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryStatus.java f78b7c3 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/FinishedLensQuery.java 6cecf7e 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java beaa72f 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/collect/ImmutableQueryCollection.java 344ff10 
>   lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java c29a1ac 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultEstimatedQueryCollection.java e3505bb 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultQueryCollection.java f9e7701 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/MutableQueryCollection.java e6e777c 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeEstimatedQueryCollection.java cdbd2ad 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeQueryCollection.java 7b43a38 
>   lens-server/src/test/java/org/apache/lens/server/query/collect/DefaultQueryCollectionTest.java 7a81e83 
>   lens-server/src/test/java/org/apache/lens/server/query/collect/QueryCollectUtil.java 51fcf00 
> 
> Diff: https://reviews.apache.org/r/37716/diff/
> 
> 
> Testing
> -------
> 
> [INFO] ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO] 
> [INFO] Lens Checkstyle Rules ............................. SUCCESS [1.847s]
> [INFO] Lens .............................................. SUCCESS [2.853s]
> [INFO] Lens API .......................................... SUCCESS [24.281s]
> [INFO] Lens API for server and extensions ................ SUCCESS [21.248s]
> [INFO] Lens Cube ......................................... SUCCESS [5:16.136s]
> [INFO] Lens DB storage ................................... SUCCESS [19.132s]
> [INFO] Lens Query Library ................................ SUCCESS [14.881s]
> [INFO] Lens Hive Driver .................................. SUCCESS [2:49.676s]
> [INFO] Lens Driver for JDBC .............................. SUCCESS [38.935s]
> [INFO] Lens Elastic Search Driver ........................ SUCCESS [16.642s]
> [INFO] Lens Server ....................................... SUCCESS [5:35.820s]
> [INFO] Lens client ....................................... SUCCESS [36.048s]
> [INFO] Lens CLI .......................................... SUCCESS [2:35.600s]
> [INFO] Lens Examples ..................................... SUCCESS [8.941s]
> [INFO] Lens Distribution ................................. SUCCESS [8.296s]
> [INFO] Lens ML Lib ....................................... SUCCESS [1:21.192s]
> [INFO] Lens ML Ext Distribution .......................... SUCCESS [2.212s]
> [INFO] Lens Regression ................................... SUCCESS [11.972s]
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 21:06.610s
> [INFO] Finished at: Mon Aug 24 15:26:07 UTC 2015
> [INFO] Final Memory: 193M/1887M
> [INFO] ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Akshay Goyal
> 
>

Re: Review Request 37716: LENS-753: Queue number for queries in submitted queue

Posted by Akshay Goyal <ak...@gmail.com>.


> On Aug. 27, 2015, 10:03 a.m., Rajat Khandelwal wrote:
> > lens-server-api/src/main/java/org/apache/lens/server/api/query/collect/ImmutableQueryCollection.java, line 58
> > <https://reviews.apache.org/r/37716/diff/1/?file=1048245#file1048245line58>
> >
> >     Function name and java doc not in sync. let's rename the function to reflect it's queue number.

Didn't want to expose implementation details here, so changed it to getQueryIndex. Missed updating javadoc.


> On Aug. 27, 2015, 10:03 a.m., Rajat Khandelwal wrote:
> > lens-server-api/src/main/java/org/apache/lens/server/api/query/FinishedLensQuery.java, line 207
> > <https://reviews.apache.org/r/37716/diff/1/?file=1048243#file1048243line207>
> >
> >     What does 0 mean as queue number?

Queue number 0 means the query is either in waiting or in completed state. A non zero value gives the queue number of a query.


> On Aug. 27, 2015, 10:03 a.m., Rajat Khandelwal wrote:
> > lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java, line 220
> > <https://reviews.apache.org/r/37716/diff/1/?file=1048244#file1048244line220>
> >
> >     Default queue number should be infinity or zero?

Should not matter, as we are not persisting the queue numbers. This is just to initialize querystatus.


- Akshay


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37716/#review96664
-----------------------------------------------------------


On Aug. 27, 2015, 9:49 a.m., Akshay Goyal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37716/
> -----------------------------------------------------------
> 
> (Updated Aug. 27, 2015, 9:49 a.m.)
> 
> 
> Review request for lens and Amareshwari Sriramadasu.
> 
> 
> Bugs: LENS-753
>     https://issues.apache.org/jira/browse/LENS-753
> 
> 
> Repository: lens
> 
> 
> Description
> -------
> 
> Queue number for queries in submitted queue
> 
> 
> Diffs
> -----
> 
>   lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java 9614caa 
>   lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java 7a5b177 
>   lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryStatus.java f78b7c3 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/FinishedLensQuery.java 6cecf7e 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java beaa72f 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/collect/ImmutableQueryCollection.java 344ff10 
>   lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java c29a1ac 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultEstimatedQueryCollection.java e3505bb 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultQueryCollection.java f9e7701 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/MutableQueryCollection.java e6e777c 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeEstimatedQueryCollection.java cdbd2ad 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeQueryCollection.java 7b43a38 
>   lens-server/src/test/java/org/apache/lens/server/query/collect/DefaultQueryCollectionTest.java 7a81e83 
>   lens-server/src/test/java/org/apache/lens/server/query/collect/QueryCollectUtil.java 51fcf00 
>   src/site/apt/user/cli.apt be93b8c 
> 
> Diff: https://reviews.apache.org/r/37716/diff/
> 
> 
> Testing
> -------
> 
> [INFO] ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO] 
> [INFO] Lens Checkstyle Rules ............................. SUCCESS [1.847s]
> [INFO] Lens .............................................. SUCCESS [2.853s]
> [INFO] Lens API .......................................... SUCCESS [24.281s]
> [INFO] Lens API for server and extensions ................ SUCCESS [21.248s]
> [INFO] Lens Cube ......................................... SUCCESS [5:16.136s]
> [INFO] Lens DB storage ................................... SUCCESS [19.132s]
> [INFO] Lens Query Library ................................ SUCCESS [14.881s]
> [INFO] Lens Hive Driver .................................. SUCCESS [2:49.676s]
> [INFO] Lens Driver for JDBC .............................. SUCCESS [38.935s]
> [INFO] Lens Elastic Search Driver ........................ SUCCESS [16.642s]
> [INFO] Lens Server ....................................... SUCCESS [5:35.820s]
> [INFO] Lens client ....................................... SUCCESS [36.048s]
> [INFO] Lens CLI .......................................... SUCCESS [2:35.600s]
> [INFO] Lens Examples ..................................... SUCCESS [8.941s]
> [INFO] Lens Distribution ................................. SUCCESS [8.296s]
> [INFO] Lens ML Lib ....................................... SUCCESS [1:21.192s]
> [INFO] Lens ML Ext Distribution .......................... SUCCESS [2.212s]
> [INFO] Lens Regression ................................... SUCCESS [11.972s]
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 21:06.610s
> [INFO] Finished at: Mon Aug 24 15:26:07 UTC 2015
> [INFO] Final Memory: 193M/1887M
> [INFO] ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Akshay Goyal
> 
>

Re: Review Request 37716: LENS-753: Queue number for queries in submitted queue

Posted by Rajat Khandelwal <ra...@gmail.com>.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37716/#review96664
-----------------------------------------------------------



lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java (lines 126 - 132)
<https://reviews.apache.org/r/37716/#comment152282>

    If queueNumber is not always populated, we should use `Integer` instead of `int`.



lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java (line 130)
<https://reviews.apache.org/r/37716/#comment152283>

    Also add documentations as to which values of this number are valid, which are special cases etc.



lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java (lines 215 - 219)
<https://reviews.apache.org/r/37716/#comment152284>

    This code can be refactored out, it's common in this function and the one above it.
    
    Though, I think let's not add the two new functions. `query status`, `query queueNumber`, `query progress` all are fetching the full status.



lens-server-api/src/main/java/org/apache/lens/server/api/query/FinishedLensQuery.java (line 207)
<https://reviews.apache.org/r/37716/#comment152285>

    What does 0 mean as queue number?



lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java (line 220)
<https://reviews.apache.org/r/37716/#comment152286>

    Default queue number should be infinity or zero?



lens-server-api/src/main/java/org/apache/lens/server/api/query/collect/ImmutableQueryCollection.java (line 58)
<https://reviews.apache.org/r/37716/#comment152287>

    Function name and java doc not in sync. let's rename the function to reflect it's queue number.


- Rajat Khandelwal


On Aug. 27, 2015, 3:19 p.m., Akshay Goyal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37716/
> -----------------------------------------------------------
> 
> (Updated Aug. 27, 2015, 3:19 p.m.)
> 
> 
> Review request for lens and Amareshwari Sriramadasu.
> 
> 
> Bugs: LENS-753
>     https://issues.apache.org/jira/browse/LENS-753
> 
> 
> Repository: lens
> 
> 
> Description
> -------
> 
> Queue number for queries in submitted queue
> 
> 
> Diffs
> -----
> 
>   lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java 9614caa 
>   lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java 7a5b177 
>   lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryStatus.java f78b7c3 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/FinishedLensQuery.java 6cecf7e 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java beaa72f 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/collect/ImmutableQueryCollection.java 344ff10 
>   lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java c29a1ac 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultEstimatedQueryCollection.java e3505bb 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultQueryCollection.java f9e7701 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/MutableQueryCollection.java e6e777c 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeEstimatedQueryCollection.java cdbd2ad 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeQueryCollection.java 7b43a38 
>   lens-server/src/test/java/org/apache/lens/server/query/collect/DefaultQueryCollectionTest.java 7a81e83 
>   lens-server/src/test/java/org/apache/lens/server/query/collect/QueryCollectUtil.java 51fcf00 
>   src/site/apt/user/cli.apt be93b8c 
> 
> Diff: https://reviews.apache.org/r/37716/diff/
> 
> 
> Testing
> -------
> 
> [INFO] ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO] 
> [INFO] Lens Checkstyle Rules ............................. SUCCESS [1.847s]
> [INFO] Lens .............................................. SUCCESS [2.853s]
> [INFO] Lens API .......................................... SUCCESS [24.281s]
> [INFO] Lens API for server and extensions ................ SUCCESS [21.248s]
> [INFO] Lens Cube ......................................... SUCCESS [5:16.136s]
> [INFO] Lens DB storage ................................... SUCCESS [19.132s]
> [INFO] Lens Query Library ................................ SUCCESS [14.881s]
> [INFO] Lens Hive Driver .................................. SUCCESS [2:49.676s]
> [INFO] Lens Driver for JDBC .............................. SUCCESS [38.935s]
> [INFO] Lens Elastic Search Driver ........................ SUCCESS [16.642s]
> [INFO] Lens Server ....................................... SUCCESS [5:35.820s]
> [INFO] Lens client ....................................... SUCCESS [36.048s]
> [INFO] Lens CLI .......................................... SUCCESS [2:35.600s]
> [INFO] Lens Examples ..................................... SUCCESS [8.941s]
> [INFO] Lens Distribution ................................. SUCCESS [8.296s]
> [INFO] Lens ML Lib ....................................... SUCCESS [1:21.192s]
> [INFO] Lens ML Ext Distribution .......................... SUCCESS [2.212s]
> [INFO] Lens Regression ................................... SUCCESS [11.972s]
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 21:06.610s
> [INFO] Finished at: Mon Aug 24 15:26:07 UTC 2015
> [INFO] Final Memory: 193M/1887M
> [INFO] ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Akshay Goyal
> 
>

Re: Review Request 37716: LENS-753: Queue number for queries in submitted queue

Posted by Amareshwari Sriramadasu <am...@apache.org>.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37716/#review98837
-----------------------------------------------------------

Ship it!


Ship It!

- Amareshwari Sriramadasu


On Sept. 14, 2015, 12:07 p.m., Akshay Goyal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37716/
> -----------------------------------------------------------
> 
> (Updated Sept. 14, 2015, 12:07 p.m.)
> 
> 
> Review request for lens and Amareshwari Sriramadasu.
> 
> 
> Bugs: LENS-753
>     https://issues.apache.org/jira/browse/LENS-753
> 
> 
> Repository: lens
> 
> 
> Description
> -------
> 
> Queue number for queries in submitted queue
> 
> 
> Diffs
> -----
> 
>   lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java 9614caa 
>   lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java 44c0c62 
>   lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryStatus.java f78b7c3 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/FinishedLensQuery.java 6cecf7e 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java beaa72f 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/collect/ImmutableQueryCollection.java 344ff10 
>   lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java 23cc748 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultEstimatedQueryCollection.java e3505bb 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultQueryCollection.java f9e7701 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeEstimatedQueryCollection.java cdbd2ad 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeQueryCollection.java 7b43a38 
>   lens-server/src/test/java/org/apache/lens/server/query/collect/DefaultQueryCollectionTest.java 7a81e83 
>   lens-server/src/test/java/org/apache/lens/server/query/collect/QueryCollectUtil.java 51fcf00 
> 
> Diff: https://reviews.apache.org/r/37716/diff/
> 
> 
> Testing
> -------
> 
> [INFO] ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO] 
> [INFO] Lens Checkstyle Rules ............................. SUCCESS [2.050s]
> [INFO] Lens .............................................. SUCCESS [3.139s]
> [INFO] Lens API .......................................... SUCCESS [24.528s]
> [INFO] Lens API for server and extensions ................ SUCCESS [20.310s]
> [INFO] Lens Cube ......................................... SUCCESS [5:12.292s]
> [INFO] Lens DB storage ................................... SUCCESS [19.318s]
> [INFO] Lens Query Library ................................ SUCCESS [16.207s]
> [INFO] Lens Hive Driver .................................. SUCCESS [2:50.394s]
> [INFO] Lens Driver for JDBC .............................. SUCCESS [35.664s]
> [INFO] Lens Elastic Search Driver ........................ SUCCESS [15.377s]
> [INFO] Lens Server ....................................... SUCCESS [5:44.102s]
> [INFO] Lens client ....................................... SUCCESS [38.027s]
> [INFO] Lens CLI .......................................... SUCCESS [2:37.369s]
> [INFO] Lens Examples ..................................... SUCCESS [8.288s]
> [INFO] Lens Distribution ................................. SUCCESS [8.495s]
> [INFO] Lens ML Lib ....................................... SUCCESS [1:18.983s]
> [INFO] Lens ML Ext Distribution .......................... SUCCESS [2.216s]
> [INFO] Lens Regression ................................... SUCCESS [10.034s]
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 21:07.695s
> [INFO] Finished at: Mon Sep 14 12:04:07 UTC 2015
> [INFO] Final Memory: 193M/1889M
> [INFO] ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Akshay Goyal
> 
>

Re: Review Request 37716: LENS-753: Queue number for queries in submitted queue

Posted by Akshay Goyal <ak...@gmail.com>.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37716/
-----------------------------------------------------------

(Updated Sept. 14, 2015, 12:07 p.m.)


Review request for lens and Amareshwari Sriramadasu.


Bugs: LENS-753
    https://issues.apache.org/jira/browse/LENS-753


Repository: lens


Description
-------

Queue number for queries in submitted queue


Diffs (updated)
-----

  lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java 9614caa 
  lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java 44c0c62 
  lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryStatus.java f78b7c3 
  lens-server-api/src/main/java/org/apache/lens/server/api/query/FinishedLensQuery.java 6cecf7e 
  lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java beaa72f 
  lens-server-api/src/main/java/org/apache/lens/server/api/query/collect/ImmutableQueryCollection.java 344ff10 
  lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java 23cc748 
  lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultEstimatedQueryCollection.java e3505bb 
  lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultQueryCollection.java f9e7701 
  lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeEstimatedQueryCollection.java cdbd2ad 
  lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeQueryCollection.java 7b43a38 
  lens-server/src/test/java/org/apache/lens/server/query/collect/DefaultQueryCollectionTest.java 7a81e83 
  lens-server/src/test/java/org/apache/lens/server/query/collect/QueryCollectUtil.java 51fcf00 

Diff: https://reviews.apache.org/r/37716/diff/


Testing (updated)
-------

[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO] 
[INFO] Lens Checkstyle Rules ............................. SUCCESS [2.050s]
[INFO] Lens .............................................. SUCCESS [3.139s]
[INFO] Lens API .......................................... SUCCESS [24.528s]
[INFO] Lens API for server and extensions ................ SUCCESS [20.310s]
[INFO] Lens Cube ......................................... SUCCESS [5:12.292s]
[INFO] Lens DB storage ................................... SUCCESS [19.318s]
[INFO] Lens Query Library ................................ SUCCESS [16.207s]
[INFO] Lens Hive Driver .................................. SUCCESS [2:50.394s]
[INFO] Lens Driver for JDBC .............................. SUCCESS [35.664s]
[INFO] Lens Elastic Search Driver ........................ SUCCESS [15.377s]
[INFO] Lens Server ....................................... SUCCESS [5:44.102s]
[INFO] Lens client ....................................... SUCCESS [38.027s]
[INFO] Lens CLI .......................................... SUCCESS [2:37.369s]
[INFO] Lens Examples ..................................... SUCCESS [8.288s]
[INFO] Lens Distribution ................................. SUCCESS [8.495s]
[INFO] Lens ML Lib ....................................... SUCCESS [1:18.983s]
[INFO] Lens ML Ext Distribution .......................... SUCCESS [2.216s]
[INFO] Lens Regression ................................... SUCCESS [10.034s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 21:07.695s
[INFO] Finished at: Mon Sep 14 12:04:07 UTC 2015
[INFO] Final Memory: 193M/1889M
[INFO] ------------------------------------------------------------------------


Thanks,

Akshay Goyal

Re: Review Request 37716: LENS-753: Queue number for queries in submitted queue

Posted by Akshay Goyal <ak...@gmail.com>.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37716/
-----------------------------------------------------------

(Updated Sept. 14, 2015, 9:19 a.m.)


Review request for lens and Amareshwari Sriramadasu.


Changes
-------

using correct variable name.


Bugs: LENS-753
    https://issues.apache.org/jira/browse/LENS-753


Repository: lens


Description
-------

Queue number for queries in submitted queue


Diffs (updated)
-----

  lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java 9614caa 
  lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java 44c0c62 
  lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryStatus.java f78b7c3 
  lens-server-api/src/main/java/org/apache/lens/server/api/query/FinishedLensQuery.java 6cecf7e 
  lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java beaa72f 
  lens-server-api/src/main/java/org/apache/lens/server/api/query/collect/ImmutableQueryCollection.java 344ff10 
  lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java 23cc748 
  lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultEstimatedQueryCollection.java e3505bb 
  lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultQueryCollection.java f9e7701 
  lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeEstimatedQueryCollection.java cdbd2ad 
  lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeQueryCollection.java 7b43a38 
  lens-server/src/test/java/org/apache/lens/server/query/collect/DefaultQueryCollectionTest.java 7a81e83 
  lens-server/src/test/java/org/apache/lens/server/query/collect/QueryCollectUtil.java 51fcf00 

Diff: https://reviews.apache.org/r/37716/diff/


Testing
-------

[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO] 
[INFO] Lens Checkstyle Rules ............................. SUCCESS [1.847s]
[INFO] Lens .............................................. SUCCESS [2.853s]
[INFO] Lens API .......................................... SUCCESS [24.281s]
[INFO] Lens API for server and extensions ................ SUCCESS [21.248s]
[INFO] Lens Cube ......................................... SUCCESS [5:16.136s]
[INFO] Lens DB storage ................................... SUCCESS [19.132s]
[INFO] Lens Query Library ................................ SUCCESS [14.881s]
[INFO] Lens Hive Driver .................................. SUCCESS [2:49.676s]
[INFO] Lens Driver for JDBC .............................. SUCCESS [38.935s]
[INFO] Lens Elastic Search Driver ........................ SUCCESS [16.642s]
[INFO] Lens Server ....................................... SUCCESS [5:35.820s]
[INFO] Lens client ....................................... SUCCESS [36.048s]
[INFO] Lens CLI .......................................... SUCCESS [2:35.600s]
[INFO] Lens Examples ..................................... SUCCESS [8.941s]
[INFO] Lens Distribution ................................. SUCCESS [8.296s]
[INFO] Lens ML Lib ....................................... SUCCESS [1:21.192s]
[INFO] Lens ML Ext Distribution .......................... SUCCESS [2.212s]
[INFO] Lens Regression ................................... SUCCESS [11.972s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 21:06.610s
[INFO] Finished at: Mon Aug 24 15:26:07 UTC 2015
[INFO] Final Memory: 193M/1887M
[INFO] ------------------------------------------------------------------------


Thanks,

Akshay Goyal

Re: Review Request 37716: LENS-753: Queue number for queries in submitted queue

Posted by Amareshwari Sriramadasu <am...@apache.org>.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37716/#review98828
-----------------------------------------------------------

Ship it!



lens-server/src/test/java/org/apache/lens/server/query/collect/DefaultQueryCollectionTest.java (line 94)
<https://reviews.apache.org/r/37716/#comment155434>

    Should the name of variable here be 'queuedQuery' ?


- Amareshwari Sriramadasu


On Sept. 14, 2015, 9:02 a.m., Akshay Goyal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37716/
> -----------------------------------------------------------
> 
> (Updated Sept. 14, 2015, 9:02 a.m.)
> 
> 
> Review request for lens and Amareshwari Sriramadasu.
> 
> 
> Bugs: LENS-753
>     https://issues.apache.org/jira/browse/LENS-753
> 
> 
> Repository: lens
> 
> 
> Description
> -------
> 
> Queue number for queries in submitted queue
> 
> 
> Diffs
> -----
> 
>   lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java 9614caa 
>   lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java 44c0c62 
>   lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryStatus.java f78b7c3 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/FinishedLensQuery.java 6cecf7e 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java beaa72f 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/collect/ImmutableQueryCollection.java 344ff10 
>   lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java 23cc748 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultEstimatedQueryCollection.java e3505bb 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultQueryCollection.java f9e7701 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeEstimatedQueryCollection.java cdbd2ad 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeQueryCollection.java 7b43a38 
>   lens-server/src/test/java/org/apache/lens/server/query/collect/DefaultQueryCollectionTest.java 7a81e83 
>   lens-server/src/test/java/org/apache/lens/server/query/collect/QueryCollectUtil.java 51fcf00 
> 
> Diff: https://reviews.apache.org/r/37716/diff/
> 
> 
> Testing
> -------
> 
> [INFO] ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO] 
> [INFO] Lens Checkstyle Rules ............................. SUCCESS [1.847s]
> [INFO] Lens .............................................. SUCCESS [2.853s]
> [INFO] Lens API .......................................... SUCCESS [24.281s]
> [INFO] Lens API for server and extensions ................ SUCCESS [21.248s]
> [INFO] Lens Cube ......................................... SUCCESS [5:16.136s]
> [INFO] Lens DB storage ................................... SUCCESS [19.132s]
> [INFO] Lens Query Library ................................ SUCCESS [14.881s]
> [INFO] Lens Hive Driver .................................. SUCCESS [2:49.676s]
> [INFO] Lens Driver for JDBC .............................. SUCCESS [38.935s]
> [INFO] Lens Elastic Search Driver ........................ SUCCESS [16.642s]
> [INFO] Lens Server ....................................... SUCCESS [5:35.820s]
> [INFO] Lens client ....................................... SUCCESS [36.048s]
> [INFO] Lens CLI .......................................... SUCCESS [2:35.600s]
> [INFO] Lens Examples ..................................... SUCCESS [8.941s]
> [INFO] Lens Distribution ................................. SUCCESS [8.296s]
> [INFO] Lens ML Lib ....................................... SUCCESS [1:21.192s]
> [INFO] Lens ML Ext Distribution .......................... SUCCESS [2.212s]
> [INFO] Lens Regression ................................... SUCCESS [11.972s]
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 21:06.610s
> [INFO] Finished at: Mon Aug 24 15:26:07 UTC 2015
> [INFO] Final Memory: 193M/1887M
> [INFO] ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Akshay Goyal
> 
>

Re: Review Request 37716: LENS-753: Queue number for queries in submitted queue

Posted by Akshay Goyal <ak...@gmail.com>.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37716/
-----------------------------------------------------------

(Updated Sept. 14, 2015, 9:02 a.m.)


Review request for lens and Amareshwari Sriramadasu.


Bugs: LENS-753
    https://issues.apache.org/jira/browse/LENS-753


Repository: lens


Description
-------

Queue number for queries in submitted queue


Diffs (updated)
-----

  lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java 9614caa 
  lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java 44c0c62 
  lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryStatus.java f78b7c3 
  lens-server-api/src/main/java/org/apache/lens/server/api/query/FinishedLensQuery.java 6cecf7e 
  lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java beaa72f 
  lens-server-api/src/main/java/org/apache/lens/server/api/query/collect/ImmutableQueryCollection.java 344ff10 
  lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java 23cc748 
  lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultEstimatedQueryCollection.java e3505bb 
  lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultQueryCollection.java f9e7701 
  lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeEstimatedQueryCollection.java cdbd2ad 
  lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeQueryCollection.java 7b43a38 
  lens-server/src/test/java/org/apache/lens/server/query/collect/DefaultQueryCollectionTest.java 7a81e83 
  lens-server/src/test/java/org/apache/lens/server/query/collect/QueryCollectUtil.java 51fcf00 

Diff: https://reviews.apache.org/r/37716/diff/


Testing
-------

[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO] 
[INFO] Lens Checkstyle Rules ............................. SUCCESS [1.847s]
[INFO] Lens .............................................. SUCCESS [2.853s]
[INFO] Lens API .......................................... SUCCESS [24.281s]
[INFO] Lens API for server and extensions ................ SUCCESS [21.248s]
[INFO] Lens Cube ......................................... SUCCESS [5:16.136s]
[INFO] Lens DB storage ................................... SUCCESS [19.132s]
[INFO] Lens Query Library ................................ SUCCESS [14.881s]
[INFO] Lens Hive Driver .................................. SUCCESS [2:49.676s]
[INFO] Lens Driver for JDBC .............................. SUCCESS [38.935s]
[INFO] Lens Elastic Search Driver ........................ SUCCESS [16.642s]
[INFO] Lens Server ....................................... SUCCESS [5:35.820s]
[INFO] Lens client ....................................... SUCCESS [36.048s]
[INFO] Lens CLI .......................................... SUCCESS [2:35.600s]
[INFO] Lens Examples ..................................... SUCCESS [8.941s]
[INFO] Lens Distribution ................................. SUCCESS [8.296s]
[INFO] Lens ML Lib ....................................... SUCCESS [1:21.192s]
[INFO] Lens ML Ext Distribution .......................... SUCCESS [2.212s]
[INFO] Lens Regression ................................... SUCCESS [11.972s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 21:06.610s
[INFO] Finished at: Mon Aug 24 15:26:07 UTC 2015
[INFO] Final Memory: 193M/1887M
[INFO] ------------------------------------------------------------------------


Thanks,

Akshay Goyal

Re: Review Request 37716: LENS-753: Queue number for queries in submitted queue

Posted by Amareshwari Sriramadasu <am...@apache.org>.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37716/#review98819
-----------------------------------------------------------



lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java (line 618)
<https://reviews.apache.org/r/37716/#comment155424>

    Set queue number as null for launched queries?


- Amareshwari Sriramadasu


On Sept. 14, 2015, 7:42 a.m., Akshay Goyal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37716/
> -----------------------------------------------------------
> 
> (Updated Sept. 14, 2015, 7:42 a.m.)
> 
> 
> Review request for lens and Amareshwari Sriramadasu.
> 
> 
> Bugs: LENS-753
>     https://issues.apache.org/jira/browse/LENS-753
> 
> 
> Repository: lens
> 
> 
> Description
> -------
> 
> Queue number for queries in submitted queue
> 
> 
> Diffs
> -----
> 
>   lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java 9614caa 
>   lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java 44c0c62 
>   lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryStatus.java f78b7c3 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/FinishedLensQuery.java 6cecf7e 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java beaa72f 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/collect/ImmutableQueryCollection.java 344ff10 
>   lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java 23cc748 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultEstimatedQueryCollection.java e3505bb 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultQueryCollection.java f9e7701 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeEstimatedQueryCollection.java cdbd2ad 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeQueryCollection.java 7b43a38 
>   lens-server/src/test/java/org/apache/lens/server/query/collect/DefaultQueryCollectionTest.java 7a81e83 
>   lens-server/src/test/java/org/apache/lens/server/query/collect/QueryCollectUtil.java 51fcf00 
> 
> Diff: https://reviews.apache.org/r/37716/diff/
> 
> 
> Testing
> -------
> 
> [INFO] ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO] 
> [INFO] Lens Checkstyle Rules ............................. SUCCESS [1.847s]
> [INFO] Lens .............................................. SUCCESS [2.853s]
> [INFO] Lens API .......................................... SUCCESS [24.281s]
> [INFO] Lens API for server and extensions ................ SUCCESS [21.248s]
> [INFO] Lens Cube ......................................... SUCCESS [5:16.136s]
> [INFO] Lens DB storage ................................... SUCCESS [19.132s]
> [INFO] Lens Query Library ................................ SUCCESS [14.881s]
> [INFO] Lens Hive Driver .................................. SUCCESS [2:49.676s]
> [INFO] Lens Driver for JDBC .............................. SUCCESS [38.935s]
> [INFO] Lens Elastic Search Driver ........................ SUCCESS [16.642s]
> [INFO] Lens Server ....................................... SUCCESS [5:35.820s]
> [INFO] Lens client ....................................... SUCCESS [36.048s]
> [INFO] Lens CLI .......................................... SUCCESS [2:35.600s]
> [INFO] Lens Examples ..................................... SUCCESS [8.941s]
> [INFO] Lens Distribution ................................. SUCCESS [8.296s]
> [INFO] Lens ML Lib ....................................... SUCCESS [1:21.192s]
> [INFO] Lens ML Ext Distribution .......................... SUCCESS [2.212s]
> [INFO] Lens Regression ................................... SUCCESS [11.972s]
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 21:06.610s
> [INFO] Finished at: Mon Aug 24 15:26:07 UTC 2015
> [INFO] Final Memory: 193M/1887M
> [INFO] ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Akshay Goyal
> 
>

Re: Review Request 37716: LENS-753: Queue number for queries in submitted queue

Posted by Akshay Goyal <ak...@gmail.com>.


> On Sept. 14, 2015, 8:24 a.m., Rajat Khandelwal wrote:
> > lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java, lines 164-179
> > <https://reviews.apache.org/r/37716/diff/5/?file=1072235#file1072235line164>
> >
> >     Can we reuse status.toString here? Both have similar code snippets. Maybe the toString will need some modifications to be generic enough to be usable here.

Done.


- Akshay


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37716/#review98823
-----------------------------------------------------------


On Sept. 14, 2015, 9:02 a.m., Akshay Goyal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37716/
> -----------------------------------------------------------
> 
> (Updated Sept. 14, 2015, 9:02 a.m.)
> 
> 
> Review request for lens and Amareshwari Sriramadasu.
> 
> 
> Bugs: LENS-753
>     https://issues.apache.org/jira/browse/LENS-753
> 
> 
> Repository: lens
> 
> 
> Description
> -------
> 
> Queue number for queries in submitted queue
> 
> 
> Diffs
> -----
> 
>   lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java 9614caa 
>   lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java 44c0c62 
>   lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryStatus.java f78b7c3 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/FinishedLensQuery.java 6cecf7e 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java beaa72f 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/collect/ImmutableQueryCollection.java 344ff10 
>   lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java 23cc748 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultEstimatedQueryCollection.java e3505bb 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultQueryCollection.java f9e7701 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeEstimatedQueryCollection.java cdbd2ad 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeQueryCollection.java 7b43a38 
>   lens-server/src/test/java/org/apache/lens/server/query/collect/DefaultQueryCollectionTest.java 7a81e83 
>   lens-server/src/test/java/org/apache/lens/server/query/collect/QueryCollectUtil.java 51fcf00 
> 
> Diff: https://reviews.apache.org/r/37716/diff/
> 
> 
> Testing
> -------
> 
> [INFO] ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO] 
> [INFO] Lens Checkstyle Rules ............................. SUCCESS [1.847s]
> [INFO] Lens .............................................. SUCCESS [2.853s]
> [INFO] Lens API .......................................... SUCCESS [24.281s]
> [INFO] Lens API for server and extensions ................ SUCCESS [21.248s]
> [INFO] Lens Cube ......................................... SUCCESS [5:16.136s]
> [INFO] Lens DB storage ................................... SUCCESS [19.132s]
> [INFO] Lens Query Library ................................ SUCCESS [14.881s]
> [INFO] Lens Hive Driver .................................. SUCCESS [2:49.676s]
> [INFO] Lens Driver for JDBC .............................. SUCCESS [38.935s]
> [INFO] Lens Elastic Search Driver ........................ SUCCESS [16.642s]
> [INFO] Lens Server ....................................... SUCCESS [5:35.820s]
> [INFO] Lens client ....................................... SUCCESS [36.048s]
> [INFO] Lens CLI .......................................... SUCCESS [2:35.600s]
> [INFO] Lens Examples ..................................... SUCCESS [8.941s]
> [INFO] Lens Distribution ................................. SUCCESS [8.296s]
> [INFO] Lens ML Lib ....................................... SUCCESS [1:21.192s]
> [INFO] Lens ML Ext Distribution .......................... SUCCESS [2.212s]
> [INFO] Lens Regression ................................... SUCCESS [11.972s]
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 21:06.610s
> [INFO] Finished at: Mon Aug 24 15:26:07 UTC 2015
> [INFO] Final Memory: 193M/1887M
> [INFO] ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Akshay Goyal
> 
>

Re: Review Request 37716: LENS-753: Queue number for queries in submitted queue

Posted by Rajat Khandelwal <ra...@gmail.com>.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37716/#review98823
-----------------------------------------------------------



lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java (lines 164 - 179)
<https://reviews.apache.org/r/37716/#comment155431>

    Can we reuse status.toString here? Both have similar code snippets. Maybe the toString will need some modifications to be generic enough to be usable here.


- Rajat Khandelwal


On Sept. 14, 2015, 1:12 p.m., Akshay Goyal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37716/
> -----------------------------------------------------------
> 
> (Updated Sept. 14, 2015, 1:12 p.m.)
> 
> 
> Review request for lens and Amareshwari Sriramadasu.
> 
> 
> Bugs: LENS-753
>     https://issues.apache.org/jira/browse/LENS-753
> 
> 
> Repository: lens
> 
> 
> Description
> -------
> 
> Queue number for queries in submitted queue
> 
> 
> Diffs
> -----
> 
>   lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java 9614caa 
>   lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java 44c0c62 
>   lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryStatus.java f78b7c3 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/FinishedLensQuery.java 6cecf7e 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java beaa72f 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/collect/ImmutableQueryCollection.java 344ff10 
>   lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java 23cc748 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultEstimatedQueryCollection.java e3505bb 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultQueryCollection.java f9e7701 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeEstimatedQueryCollection.java cdbd2ad 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeQueryCollection.java 7b43a38 
>   lens-server/src/test/java/org/apache/lens/server/query/collect/DefaultQueryCollectionTest.java 7a81e83 
>   lens-server/src/test/java/org/apache/lens/server/query/collect/QueryCollectUtil.java 51fcf00 
> 
> Diff: https://reviews.apache.org/r/37716/diff/
> 
> 
> Testing
> -------
> 
> [INFO] ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO] 
> [INFO] Lens Checkstyle Rules ............................. SUCCESS [1.847s]
> [INFO] Lens .............................................. SUCCESS [2.853s]
> [INFO] Lens API .......................................... SUCCESS [24.281s]
> [INFO] Lens API for server and extensions ................ SUCCESS [21.248s]
> [INFO] Lens Cube ......................................... SUCCESS [5:16.136s]
> [INFO] Lens DB storage ................................... SUCCESS [19.132s]
> [INFO] Lens Query Library ................................ SUCCESS [14.881s]
> [INFO] Lens Hive Driver .................................. SUCCESS [2:49.676s]
> [INFO] Lens Driver for JDBC .............................. SUCCESS [38.935s]
> [INFO] Lens Elastic Search Driver ........................ SUCCESS [16.642s]
> [INFO] Lens Server ....................................... SUCCESS [5:35.820s]
> [INFO] Lens client ....................................... SUCCESS [36.048s]
> [INFO] Lens CLI .......................................... SUCCESS [2:35.600s]
> [INFO] Lens Examples ..................................... SUCCESS [8.941s]
> [INFO] Lens Distribution ................................. SUCCESS [8.296s]
> [INFO] Lens ML Lib ....................................... SUCCESS [1:21.192s]
> [INFO] Lens ML Ext Distribution .......................... SUCCESS [2.212s]
> [INFO] Lens Regression ................................... SUCCESS [11.972s]
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 21:06.610s
> [INFO] Finished at: Mon Aug 24 15:26:07 UTC 2015
> [INFO] Final Memory: 193M/1887M
> [INFO] ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Akshay Goyal
> 
>

Re: Review Request 37716: LENS-753: Queue number for queries in submitted queue

Posted by Akshay Goyal <ak...@gmail.com>.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37716/
-----------------------------------------------------------

(Updated Sept. 14, 2015, 7:42 a.m.)


Review request for lens and Amareshwari Sriramadasu.


Changes
-------

Review comments addressed.


Bugs: LENS-753
    https://issues.apache.org/jira/browse/LENS-753


Repository: lens


Description
-------

Queue number for queries in submitted queue


Diffs (updated)
-----

  lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java 9614caa 
  lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java 44c0c62 
  lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryStatus.java f78b7c3 
  lens-server-api/src/main/java/org/apache/lens/server/api/query/FinishedLensQuery.java 6cecf7e 
  lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java beaa72f 
  lens-server-api/src/main/java/org/apache/lens/server/api/query/collect/ImmutableQueryCollection.java 344ff10 
  lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java 23cc748 
  lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultEstimatedQueryCollection.java e3505bb 
  lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultQueryCollection.java f9e7701 
  lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeEstimatedQueryCollection.java cdbd2ad 
  lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeQueryCollection.java 7b43a38 
  lens-server/src/test/java/org/apache/lens/server/query/collect/DefaultQueryCollectionTest.java 7a81e83 
  lens-server/src/test/java/org/apache/lens/server/query/collect/QueryCollectUtil.java 51fcf00 

Diff: https://reviews.apache.org/r/37716/diff/


Testing
-------

[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO] 
[INFO] Lens Checkstyle Rules ............................. SUCCESS [1.847s]
[INFO] Lens .............................................. SUCCESS [2.853s]
[INFO] Lens API .......................................... SUCCESS [24.281s]
[INFO] Lens API for server and extensions ................ SUCCESS [21.248s]
[INFO] Lens Cube ......................................... SUCCESS [5:16.136s]
[INFO] Lens DB storage ................................... SUCCESS [19.132s]
[INFO] Lens Query Library ................................ SUCCESS [14.881s]
[INFO] Lens Hive Driver .................................. SUCCESS [2:49.676s]
[INFO] Lens Driver for JDBC .............................. SUCCESS [38.935s]
[INFO] Lens Elastic Search Driver ........................ SUCCESS [16.642s]
[INFO] Lens Server ....................................... SUCCESS [5:35.820s]
[INFO] Lens client ....................................... SUCCESS [36.048s]
[INFO] Lens CLI .......................................... SUCCESS [2:35.600s]
[INFO] Lens Examples ..................................... SUCCESS [8.941s]
[INFO] Lens Distribution ................................. SUCCESS [8.296s]
[INFO] Lens ML Lib ....................................... SUCCESS [1:21.192s]
[INFO] Lens ML Ext Distribution .......................... SUCCESS [2.212s]
[INFO] Lens Regression ................................... SUCCESS [11.972s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 21:06.610s
[INFO] Finished at: Mon Aug 24 15:26:07 UTC 2015
[INFO] Final Memory: 193M/1887M
[INFO] ------------------------------------------------------------------------


Thanks,

Akshay Goyal

Re: Review Request 37716: LENS-753: Queue number for queries in submitted queue

Posted by Akshay Goyal <ak...@gmail.com>.


> On Sept. 14, 2015, 6:06 a.m., Amareshwari Sriramadasu wrote:
> > lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java, line 811
> > <https://reviews.apache.org/r/37716/diff/4/?file=1072081#file1072081line811>
> >
> >     Should we set only if index returned is a valid index?
> >     
> >     Index may not be returned when it is in priority queue. And returning previos queue position would make sense in those cases.

Makes sense. Updated.


- Akshay


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37716/#review98814
-----------------------------------------------------------


On Sept. 14, 2015, 7:42 a.m., Akshay Goyal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37716/
> -----------------------------------------------------------
> 
> (Updated Sept. 14, 2015, 7:42 a.m.)
> 
> 
> Review request for lens and Amareshwari Sriramadasu.
> 
> 
> Bugs: LENS-753
>     https://issues.apache.org/jira/browse/LENS-753
> 
> 
> Repository: lens
> 
> 
> Description
> -------
> 
> Queue number for queries in submitted queue
> 
> 
> Diffs
> -----
> 
>   lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java 9614caa 
>   lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java 44c0c62 
>   lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryStatus.java f78b7c3 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/FinishedLensQuery.java 6cecf7e 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java beaa72f 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/collect/ImmutableQueryCollection.java 344ff10 
>   lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java 23cc748 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultEstimatedQueryCollection.java e3505bb 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultQueryCollection.java f9e7701 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeEstimatedQueryCollection.java cdbd2ad 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeQueryCollection.java 7b43a38 
>   lens-server/src/test/java/org/apache/lens/server/query/collect/DefaultQueryCollectionTest.java 7a81e83 
>   lens-server/src/test/java/org/apache/lens/server/query/collect/QueryCollectUtil.java 51fcf00 
> 
> Diff: https://reviews.apache.org/r/37716/diff/
> 
> 
> Testing
> -------
> 
> [INFO] ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO] 
> [INFO] Lens Checkstyle Rules ............................. SUCCESS [1.847s]
> [INFO] Lens .............................................. SUCCESS [2.853s]
> [INFO] Lens API .......................................... SUCCESS [24.281s]
> [INFO] Lens API for server and extensions ................ SUCCESS [21.248s]
> [INFO] Lens Cube ......................................... SUCCESS [5:16.136s]
> [INFO] Lens DB storage ................................... SUCCESS [19.132s]
> [INFO] Lens Query Library ................................ SUCCESS [14.881s]
> [INFO] Lens Hive Driver .................................. SUCCESS [2:49.676s]
> [INFO] Lens Driver for JDBC .............................. SUCCESS [38.935s]
> [INFO] Lens Elastic Search Driver ........................ SUCCESS [16.642s]
> [INFO] Lens Server ....................................... SUCCESS [5:35.820s]
> [INFO] Lens client ....................................... SUCCESS [36.048s]
> [INFO] Lens CLI .......................................... SUCCESS [2:35.600s]
> [INFO] Lens Examples ..................................... SUCCESS [8.941s]
> [INFO] Lens Distribution ................................. SUCCESS [8.296s]
> [INFO] Lens ML Lib ....................................... SUCCESS [1:21.192s]
> [INFO] Lens ML Ext Distribution .......................... SUCCESS [2.212s]
> [INFO] Lens Regression ................................... SUCCESS [11.972s]
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 21:06.610s
> [INFO] Finished at: Mon Aug 24 15:26:07 UTC 2015
> [INFO] Final Memory: 193M/1887M
> [INFO] ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Akshay Goyal
> 
>

Re: Review Request 37716: LENS-753: Queue number for queries in submitted queue

Posted by Amareshwari Sriramadasu <am...@apache.org>.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37716/#review98814
-----------------------------------------------------------



lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java (line 174)
<https://reviews.apache.org/r/37716/#comment155412>

    Have text as "Position in queue"



lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java (line 174)
<https://reviews.apache.org/r/37716/#comment155413>

    Have text as "Position in queue" here as well.



lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java (line 811)
<https://reviews.apache.org/r/37716/#comment155416>

    Should we set only if index returned is a valid index?
    
    Index may not be returned when it is in priority queue. And returning previos queue position would make sense in those cases.



lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java (line 1013)
<https://reviews.apache.org/r/37716/#comment155417>

    We need to check if any of the semantics change now, wrt this change.


- Amareshwari Sriramadasu


On Sept. 14, 2015, 4:57 a.m., Akshay Goyal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37716/
> -----------------------------------------------------------
> 
> (Updated Sept. 14, 2015, 4:57 a.m.)
> 
> 
> Review request for lens and Amareshwari Sriramadasu.
> 
> 
> Bugs: LENS-753
>     https://issues.apache.org/jira/browse/LENS-753
> 
> 
> Repository: lens
> 
> 
> Description
> -------
> 
> Queue number for queries in submitted queue
> 
> 
> Diffs
> -----
> 
>   lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java 9614caa 
>   lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java 44c0c62 
>   lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryStatus.java f78b7c3 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/FinishedLensQuery.java 6cecf7e 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java beaa72f 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/collect/ImmutableQueryCollection.java 344ff10 
>   lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java 23cc748 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultEstimatedQueryCollection.java e3505bb 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultQueryCollection.java f9e7701 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeEstimatedQueryCollection.java cdbd2ad 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeQueryCollection.java 7b43a38 
>   lens-server/src/test/java/org/apache/lens/server/query/collect/DefaultQueryCollectionTest.java 7a81e83 
>   lens-server/src/test/java/org/apache/lens/server/query/collect/QueryCollectUtil.java 51fcf00 
> 
> Diff: https://reviews.apache.org/r/37716/diff/
> 
> 
> Testing
> -------
> 
> [INFO] ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO] 
> [INFO] Lens Checkstyle Rules ............................. SUCCESS [1.847s]
> [INFO] Lens .............................................. SUCCESS [2.853s]
> [INFO] Lens API .......................................... SUCCESS [24.281s]
> [INFO] Lens API for server and extensions ................ SUCCESS [21.248s]
> [INFO] Lens Cube ......................................... SUCCESS [5:16.136s]
> [INFO] Lens DB storage ................................... SUCCESS [19.132s]
> [INFO] Lens Query Library ................................ SUCCESS [14.881s]
> [INFO] Lens Hive Driver .................................. SUCCESS [2:49.676s]
> [INFO] Lens Driver for JDBC .............................. SUCCESS [38.935s]
> [INFO] Lens Elastic Search Driver ........................ SUCCESS [16.642s]
> [INFO] Lens Server ....................................... SUCCESS [5:35.820s]
> [INFO] Lens client ....................................... SUCCESS [36.048s]
> [INFO] Lens CLI .......................................... SUCCESS [2:35.600s]
> [INFO] Lens Examples ..................................... SUCCESS [8.941s]
> [INFO] Lens Distribution ................................. SUCCESS [8.296s]
> [INFO] Lens ML Lib ....................................... SUCCESS [1:21.192s]
> [INFO] Lens ML Ext Distribution .......................... SUCCESS [2.212s]
> [INFO] Lens Regression ................................... SUCCESS [11.972s]
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 21:06.610s
> [INFO] Finished at: Mon Aug 24 15:26:07 UTC 2015
> [INFO] Final Memory: 193M/1887M
> [INFO] ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Akshay Goyal
> 
>

Re: Review Request 37716: LENS-753: Queue number for queries in submitted queue

Posted by Akshay Goyal <ak...@gmail.com>.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37716/
-----------------------------------------------------------

(Updated Sept. 14, 2015, 4:57 a.m.)


Review request for lens and Amareshwari Sriramadasu.


Changes
-------

fixed enunciate error


Bugs: LENS-753
    https://issues.apache.org/jira/browse/LENS-753


Repository: lens


Description
-------

Queue number for queries in submitted queue


Diffs (updated)
-----

  lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java 9614caa 
  lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java 44c0c62 
  lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryStatus.java f78b7c3 
  lens-server-api/src/main/java/org/apache/lens/server/api/query/FinishedLensQuery.java 6cecf7e 
  lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java beaa72f 
  lens-server-api/src/main/java/org/apache/lens/server/api/query/collect/ImmutableQueryCollection.java 344ff10 
  lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java 23cc748 
  lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultEstimatedQueryCollection.java e3505bb 
  lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultQueryCollection.java f9e7701 
  lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeEstimatedQueryCollection.java cdbd2ad 
  lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeQueryCollection.java 7b43a38 
  lens-server/src/test/java/org/apache/lens/server/query/collect/DefaultQueryCollectionTest.java 7a81e83 
  lens-server/src/test/java/org/apache/lens/server/query/collect/QueryCollectUtil.java 51fcf00 

Diff: https://reviews.apache.org/r/37716/diff/


Testing
-------

[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO] 
[INFO] Lens Checkstyle Rules ............................. SUCCESS [1.847s]
[INFO] Lens .............................................. SUCCESS [2.853s]
[INFO] Lens API .......................................... SUCCESS [24.281s]
[INFO] Lens API for server and extensions ................ SUCCESS [21.248s]
[INFO] Lens Cube ......................................... SUCCESS [5:16.136s]
[INFO] Lens DB storage ................................... SUCCESS [19.132s]
[INFO] Lens Query Library ................................ SUCCESS [14.881s]
[INFO] Lens Hive Driver .................................. SUCCESS [2:49.676s]
[INFO] Lens Driver for JDBC .............................. SUCCESS [38.935s]
[INFO] Lens Elastic Search Driver ........................ SUCCESS [16.642s]
[INFO] Lens Server ....................................... SUCCESS [5:35.820s]
[INFO] Lens client ....................................... SUCCESS [36.048s]
[INFO] Lens CLI .......................................... SUCCESS [2:35.600s]
[INFO] Lens Examples ..................................... SUCCESS [8.941s]
[INFO] Lens Distribution ................................. SUCCESS [8.296s]
[INFO] Lens ML Lib ....................................... SUCCESS [1:21.192s]
[INFO] Lens ML Ext Distribution .......................... SUCCESS [2.212s]
[INFO] Lens Regression ................................... SUCCESS [11.972s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 21:06.610s
[INFO] Finished at: Mon Aug 24 15:26:07 UTC 2015
[INFO] Final Memory: 193M/1887M
[INFO] ------------------------------------------------------------------------


Thanks,

Akshay Goyal

Re: Review Request 37716: LENS-753: Queue number for queries in submitted queue

Posted by Akshay Goyal <ak...@gmail.com>.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37716/
-----------------------------------------------------------

(Updated Sept. 13, 2015, 1:14 p.m.)


Review request for lens and Amareshwari Sriramadasu.


Changes
-------

changes for retrieving queue number from waiting queries collection.


Bugs: LENS-753
    https://issues.apache.org/jira/browse/LENS-753


Repository: lens


Description
-------

Queue number for queries in submitted queue


Diffs (updated)
-----

  lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java 9614caa 
  lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java 44c0c62 
  lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryStatus.java f78b7c3 
  lens-server-api/src/main/java/org/apache/lens/server/api/query/FinishedLensQuery.java 6cecf7e 
  lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java beaa72f 
  lens-server-api/src/main/java/org/apache/lens/server/api/query/collect/ImmutableQueryCollection.java 344ff10 
  lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java 23cc748 
  lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultEstimatedQueryCollection.java e3505bb 
  lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultQueryCollection.java f9e7701 
  lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeEstimatedQueryCollection.java cdbd2ad 
  lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeQueryCollection.java 7b43a38 
  lens-server/src/test/java/org/apache/lens/server/query/collect/DefaultQueryCollectionTest.java 7a81e83 
  lens-server/src/test/java/org/apache/lens/server/query/collect/QueryCollectUtil.java 51fcf00 

Diff: https://reviews.apache.org/r/37716/diff/


Testing
-------

[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO] 
[INFO] Lens Checkstyle Rules ............................. SUCCESS [1.847s]
[INFO] Lens .............................................. SUCCESS [2.853s]
[INFO] Lens API .......................................... SUCCESS [24.281s]
[INFO] Lens API for server and extensions ................ SUCCESS [21.248s]
[INFO] Lens Cube ......................................... SUCCESS [5:16.136s]
[INFO] Lens DB storage ................................... SUCCESS [19.132s]
[INFO] Lens Query Library ................................ SUCCESS [14.881s]
[INFO] Lens Hive Driver .................................. SUCCESS [2:49.676s]
[INFO] Lens Driver for JDBC .............................. SUCCESS [38.935s]
[INFO] Lens Elastic Search Driver ........................ SUCCESS [16.642s]
[INFO] Lens Server ....................................... SUCCESS [5:35.820s]
[INFO] Lens client ....................................... SUCCESS [36.048s]
[INFO] Lens CLI .......................................... SUCCESS [2:35.600s]
[INFO] Lens Examples ..................................... SUCCESS [8.941s]
[INFO] Lens Distribution ................................. SUCCESS [8.296s]
[INFO] Lens ML Lib ....................................... SUCCESS [1:21.192s]
[INFO] Lens ML Ext Distribution .......................... SUCCESS [2.212s]
[INFO] Lens Regression ................................... SUCCESS [11.972s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 21:06.610s
[INFO] Finished at: Mon Aug 24 15:26:07 UTC 2015
[INFO] Final Memory: 193M/1887M
[INFO] ------------------------------------------------------------------------


Thanks,

Akshay Goyal

Re: Review Request 37716: LENS-753: Queue number for queries in submitted queue

Posted by Amareshwari Sriramadasu <am...@apache.org>.


> On Aug. 27, 2015, 3:48 p.m., Amareshwari Sriramadasu wrote:
> > lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java, line 181
> > <https://reviews.apache.org/r/37716/diff/2/?file=1055973#file1055973line181>
> >
> >     Seems queue number is added for queries in RUNNING state instead of queued state

If this is running position number, lets rename it to be the same - running position or something similar


- Amareshwari


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37716/#review96687
-----------------------------------------------------------


On Aug. 27, 2015, 1:34 p.m., Akshay Goyal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37716/
> -----------------------------------------------------------
> 
> (Updated Aug. 27, 2015, 1:34 p.m.)
> 
> 
> Review request for lens and Amareshwari Sriramadasu.
> 
> 
> Bugs: LENS-753
>     https://issues.apache.org/jira/browse/LENS-753
> 
> 
> Repository: lens
> 
> 
> Description
> -------
> 
> Queue number for queries in submitted queue
> 
> 
> Diffs
> -----
> 
>   lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java 9614caa 
>   lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java 7a5b177 
>   lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryStatus.java f78b7c3 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/FinishedLensQuery.java 6cecf7e 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java beaa72f 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/collect/ImmutableQueryCollection.java 344ff10 
>   lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java c29a1ac 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultEstimatedQueryCollection.java e3505bb 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultQueryCollection.java f9e7701 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/MutableQueryCollection.java e6e777c 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeEstimatedQueryCollection.java cdbd2ad 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeQueryCollection.java 7b43a38 
>   lens-server/src/test/java/org/apache/lens/server/query/collect/DefaultQueryCollectionTest.java 7a81e83 
>   lens-server/src/test/java/org/apache/lens/server/query/collect/QueryCollectUtil.java 51fcf00 
> 
> Diff: https://reviews.apache.org/r/37716/diff/
> 
> 
> Testing
> -------
> 
> [INFO] ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO] 
> [INFO] Lens Checkstyle Rules ............................. SUCCESS [1.847s]
> [INFO] Lens .............................................. SUCCESS [2.853s]
> [INFO] Lens API .......................................... SUCCESS [24.281s]
> [INFO] Lens API for server and extensions ................ SUCCESS [21.248s]
> [INFO] Lens Cube ......................................... SUCCESS [5:16.136s]
> [INFO] Lens DB storage ................................... SUCCESS [19.132s]
> [INFO] Lens Query Library ................................ SUCCESS [14.881s]
> [INFO] Lens Hive Driver .................................. SUCCESS [2:49.676s]
> [INFO] Lens Driver for JDBC .............................. SUCCESS [38.935s]
> [INFO] Lens Elastic Search Driver ........................ SUCCESS [16.642s]
> [INFO] Lens Server ....................................... SUCCESS [5:35.820s]
> [INFO] Lens client ....................................... SUCCESS [36.048s]
> [INFO] Lens CLI .......................................... SUCCESS [2:35.600s]
> [INFO] Lens Examples ..................................... SUCCESS [8.941s]
> [INFO] Lens Distribution ................................. SUCCESS [8.296s]
> [INFO] Lens ML Lib ....................................... SUCCESS [1:21.192s]
> [INFO] Lens ML Ext Distribution .......................... SUCCESS [2.212s]
> [INFO] Lens Regression ................................... SUCCESS [11.972s]
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 21:06.610s
> [INFO] Finished at: Mon Aug 24 15:26:07 UTC 2015
> [INFO] Final Memory: 193M/1887M
> [INFO] ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Akshay Goyal
> 
>

Re: Review Request 37716: LENS-753: Queue number for queries in submitted queue

Posted by Akshay Goyal <ak...@gmail.com>.


> On Aug. 27, 2015, 3:48 p.m., Amareshwari Sriramadasu wrote:
> > lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java, line 181
> > <https://reviews.apache.org/r/37716/diff/2/?file=1055973#file1055973line181>
> >
> >     Seems queue number is added for queries in RUNNING state instead of queued state
> 
> Amareshwari Sriramadasu wrote:
>     If this is running position number, lets rename it to be the same - running position or something similar

changed it only for queued state.


> On Aug. 27, 2015, 3:48 p.m., Amareshwari Sriramadasu wrote:
> > lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryStatus.java, line 189
> > <https://reviews.apache.org/r/37716/diff/2/?file=1055975#file1055975line189>
> >
> >     what is queue number in driver status?

Not required. Removed.


- Akshay


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37716/#review96687
-----------------------------------------------------------


On Sept. 13, 2015, 1:14 p.m., Akshay Goyal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37716/
> -----------------------------------------------------------
> 
> (Updated Sept. 13, 2015, 1:14 p.m.)
> 
> 
> Review request for lens and Amareshwari Sriramadasu.
> 
> 
> Bugs: LENS-753
>     https://issues.apache.org/jira/browse/LENS-753
> 
> 
> Repository: lens
> 
> 
> Description
> -------
> 
> Queue number for queries in submitted queue
> 
> 
> Diffs
> -----
> 
>   lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java 9614caa 
>   lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java 44c0c62 
>   lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryStatus.java f78b7c3 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/FinishedLensQuery.java 6cecf7e 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java beaa72f 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/collect/ImmutableQueryCollection.java 344ff10 
>   lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java 23cc748 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultEstimatedQueryCollection.java e3505bb 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultQueryCollection.java f9e7701 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeEstimatedQueryCollection.java cdbd2ad 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeQueryCollection.java 7b43a38 
>   lens-server/src/test/java/org/apache/lens/server/query/collect/DefaultQueryCollectionTest.java 7a81e83 
>   lens-server/src/test/java/org/apache/lens/server/query/collect/QueryCollectUtil.java 51fcf00 
> 
> Diff: https://reviews.apache.org/r/37716/diff/
> 
> 
> Testing
> -------
> 
> [INFO] ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO] 
> [INFO] Lens Checkstyle Rules ............................. SUCCESS [1.847s]
> [INFO] Lens .............................................. SUCCESS [2.853s]
> [INFO] Lens API .......................................... SUCCESS [24.281s]
> [INFO] Lens API for server and extensions ................ SUCCESS [21.248s]
> [INFO] Lens Cube ......................................... SUCCESS [5:16.136s]
> [INFO] Lens DB storage ................................... SUCCESS [19.132s]
> [INFO] Lens Query Library ................................ SUCCESS [14.881s]
> [INFO] Lens Hive Driver .................................. SUCCESS [2:49.676s]
> [INFO] Lens Driver for JDBC .............................. SUCCESS [38.935s]
> [INFO] Lens Elastic Search Driver ........................ SUCCESS [16.642s]
> [INFO] Lens Server ....................................... SUCCESS [5:35.820s]
> [INFO] Lens client ....................................... SUCCESS [36.048s]
> [INFO] Lens CLI .......................................... SUCCESS [2:35.600s]
> [INFO] Lens Examples ..................................... SUCCESS [8.941s]
> [INFO] Lens Distribution ................................. SUCCESS [8.296s]
> [INFO] Lens ML Lib ....................................... SUCCESS [1:21.192s]
> [INFO] Lens ML Ext Distribution .......................... SUCCESS [2.212s]
> [INFO] Lens Regression ................................... SUCCESS [11.972s]
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 21:06.610s
> [INFO] Finished at: Mon Aug 24 15:26:07 UTC 2015
> [INFO] Final Memory: 193M/1887M
> [INFO] ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Akshay Goyal
> 
>

Re: Review Request 37716: LENS-753: Queue number for queries in submitted queue

Posted by Amareshwari Sriramadasu <am...@apache.org>.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37716/#review96687
-----------------------------------------------------------



lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java (line 181)
<https://reviews.apache.org/r/37716/#comment152308>

    Seems queue number is added for queries in RUNNING state instead of queued state



lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryStatus.java (line 189)
<https://reviews.apache.org/r/37716/#comment152309>

    what is queue number in driver status?



lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java (line 618)
<https://reviews.apache.org/r/37716/#comment152310>

    why would a launched query have queue number?


- Amareshwari Sriramadasu


On Aug. 27, 2015, 1:34 p.m., Akshay Goyal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37716/
> -----------------------------------------------------------
> 
> (Updated Aug. 27, 2015, 1:34 p.m.)
> 
> 
> Review request for lens and Amareshwari Sriramadasu.
> 
> 
> Bugs: LENS-753
>     https://issues.apache.org/jira/browse/LENS-753
> 
> 
> Repository: lens
> 
> 
> Description
> -------
> 
> Queue number for queries in submitted queue
> 
> 
> Diffs
> -----
> 
>   lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java 9614caa 
>   lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java 7a5b177 
>   lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryStatus.java f78b7c3 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/FinishedLensQuery.java 6cecf7e 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java beaa72f 
>   lens-server-api/src/main/java/org/apache/lens/server/api/query/collect/ImmutableQueryCollection.java 344ff10 
>   lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java c29a1ac 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultEstimatedQueryCollection.java e3505bb 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultQueryCollection.java f9e7701 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/MutableQueryCollection.java e6e777c 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeEstimatedQueryCollection.java cdbd2ad 
>   lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeQueryCollection.java 7b43a38 
>   lens-server/src/test/java/org/apache/lens/server/query/collect/DefaultQueryCollectionTest.java 7a81e83 
>   lens-server/src/test/java/org/apache/lens/server/query/collect/QueryCollectUtil.java 51fcf00 
> 
> Diff: https://reviews.apache.org/r/37716/diff/
> 
> 
> Testing
> -------
> 
> [INFO] ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO] 
> [INFO] Lens Checkstyle Rules ............................. SUCCESS [1.847s]
> [INFO] Lens .............................................. SUCCESS [2.853s]
> [INFO] Lens API .......................................... SUCCESS [24.281s]
> [INFO] Lens API for server and extensions ................ SUCCESS [21.248s]
> [INFO] Lens Cube ......................................... SUCCESS [5:16.136s]
> [INFO] Lens DB storage ................................... SUCCESS [19.132s]
> [INFO] Lens Query Library ................................ SUCCESS [14.881s]
> [INFO] Lens Hive Driver .................................. SUCCESS [2:49.676s]
> [INFO] Lens Driver for JDBC .............................. SUCCESS [38.935s]
> [INFO] Lens Elastic Search Driver ........................ SUCCESS [16.642s]
> [INFO] Lens Server ....................................... SUCCESS [5:35.820s]
> [INFO] Lens client ....................................... SUCCESS [36.048s]
> [INFO] Lens CLI .......................................... SUCCESS [2:35.600s]
> [INFO] Lens Examples ..................................... SUCCESS [8.941s]
> [INFO] Lens Distribution ................................. SUCCESS [8.296s]
> [INFO] Lens ML Lib ....................................... SUCCESS [1:21.192s]
> [INFO] Lens ML Ext Distribution .......................... SUCCESS [2.212s]
> [INFO] Lens Regression ................................... SUCCESS [11.972s]
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 21:06.610s
> [INFO] Finished at: Mon Aug 24 15:26:07 UTC 2015
> [INFO] Final Memory: 193M/1887M
> [INFO] ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Akshay Goyal
> 
>

Re: Review Request 37716: LENS-753: Queue number for queries in submitted queue

Posted by Akshay Goyal <ak...@gmail.com>.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37716/
-----------------------------------------------------------

(Updated Aug. 27, 2015, 1:34 p.m.)


Review request for lens and Amareshwari Sriramadasu.


Changes
-------

review changes


Bugs: LENS-753
    https://issues.apache.org/jira/browse/LENS-753


Repository: lens


Description
-------

Queue number for queries in submitted queue


Diffs (updated)
-----

  lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java 9614caa 
  lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java 7a5b177 
  lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryStatus.java f78b7c3 
  lens-server-api/src/main/java/org/apache/lens/server/api/query/FinishedLensQuery.java 6cecf7e 
  lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java beaa72f 
  lens-server-api/src/main/java/org/apache/lens/server/api/query/collect/ImmutableQueryCollection.java 344ff10 
  lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java c29a1ac 
  lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultEstimatedQueryCollection.java e3505bb 
  lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultQueryCollection.java f9e7701 
  lens-server/src/main/java/org/apache/lens/server/query/collect/MutableQueryCollection.java e6e777c 
  lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeEstimatedQueryCollection.java cdbd2ad 
  lens-server/src/main/java/org/apache/lens/server/query/collect/ThreadSafeQueryCollection.java 7b43a38 
  lens-server/src/test/java/org/apache/lens/server/query/collect/DefaultQueryCollectionTest.java 7a81e83 
  lens-server/src/test/java/org/apache/lens/server/query/collect/QueryCollectUtil.java 51fcf00 

Diff: https://reviews.apache.org/r/37716/diff/


Testing
-------

[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO] 
[INFO] Lens Checkstyle Rules ............................. SUCCESS [1.847s]
[INFO] Lens .............................................. SUCCESS [2.853s]
[INFO] Lens API .......................................... SUCCESS [24.281s]
[INFO] Lens API for server and extensions ................ SUCCESS [21.248s]
[INFO] Lens Cube ......................................... SUCCESS [5:16.136s]
[INFO] Lens DB storage ................................... SUCCESS [19.132s]
[INFO] Lens Query Library ................................ SUCCESS [14.881s]
[INFO] Lens Hive Driver .................................. SUCCESS [2:49.676s]
[INFO] Lens Driver for JDBC .............................. SUCCESS [38.935s]
[INFO] Lens Elastic Search Driver ........................ SUCCESS [16.642s]
[INFO] Lens Server ....................................... SUCCESS [5:35.820s]
[INFO] Lens client ....................................... SUCCESS [36.048s]
[INFO] Lens CLI .......................................... SUCCESS [2:35.600s]
[INFO] Lens Examples ..................................... SUCCESS [8.941s]
[INFO] Lens Distribution ................................. SUCCESS [8.296s]
[INFO] Lens ML Lib ....................................... SUCCESS [1:21.192s]
[INFO] Lens ML Ext Distribution .......................... SUCCESS [2.212s]
[INFO] Lens Regression ................................... SUCCESS [11.972s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 21:06.610s
[INFO] Finished at: Mon Aug 24 15:26:07 UTC 2015
[INFO] Final Memory: 193M/1887M
[INFO] ------------------------------------------------------------------------


Thanks,

Akshay Goyal