> On June 8, 2016, 1:56 a.m., Puneet Gupta wrote:
> > lens-regression/src/test/java/org/apache/lens/regression/throttling/Throttling.java,
> > line 511
> > <https://reviews.apache.org/r/47593/diff/3/?file=1408203#file1408203line511>
> >
> > Can we remove null check and add a final else fail("Unexpected
> > Priority").
Firing queries which have priority other than low and very_low as well, so
can't put else fail ("unexpected prioirty")
> On June 8, 2016, 1:56 a.m., Puneet Gupta wrote:
> > lens-regression/src/test/java/org/apache/lens/regression/throttling/Throttling.java,
> > line 519
> > <https://reviews.apache.org/r/47593/diff/3/?file=1408203#file1408203line519>
> >
> > can we add a final else fail("Unexpected Queue").
Firing queries with default queue as well, so can't put else fail ("unexpected
queue")
- Archana
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47593/#review136578
-----------------------------------------------------------
On June 7, 2016, 6:33 a.m., Archana H wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/47593/
> -----------------------------------------------------------
>
> (Updated June 7, 2016, 6:33 a.m.)
>
>
> Review request for lens, Deepak Barr, Rajat Khandelwal, and Puneet Gupta.
>
>
> Repository: lens
>
>
> Description
> -------
>
> Throttling tests to regression
>
>
> Diffs
> -----
>
> lens-regression/pom.xml b9bee97
> lens-regression/src/main/java/org/apache/lens/regression/util/Util.java
> 1a538da
>
> lens-regression/src/test/java/org/apache/lens/regression/client/KillQueryTests.java
> e588c1b
>
> lens-regression/src/test/java/org/apache/lens/regression/throttling/ITCostTests.java
> 4da4c3a
>
> lens-regression/src/test/java/org/apache/lens/regression/throttling/Throttling.java
> PRE-CREATION
>
> Diff: https://reviews.apache.org/r/47593/diff/
>
>
> Testing
> -------
>
> [INFO]
> ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO]
> [INFO] Lens Checkstyle Rules ............................. SUCCESS [1.563s]
> [INFO] Lens .............................................. SUCCESS [3.553s]
> [INFO] Lens API .......................................... SUCCESS [20.758s]
> [INFO] Lens API for server and extensions ................ SUCCESS [16.842s]
> [INFO] Lens Cube ......................................... SUCCESS
> [14:01.494s]
> [INFO] Lens DB storage ................................... SUCCESS [17.028s]
> [INFO] Lens Query Library ................................ SUCCESS [18.060s]
> [INFO] Lens Hive Driver .................................. SUCCESS [1:33.480s]
> [INFO] Lens Driver for JDBC .............................. SUCCESS [38.171s]
> [INFO] Lens Elastic Search Driver ........................ SUCCESS [23.389s]
> [INFO] Lens Server ....................................... SUCCESS [8:49.961s]
> [INFO] Lens client ....................................... SUCCESS [1:29.573s]
> [INFO] Lens CLI .......................................... SUCCESS [1:18.608s]
> [INFO] Lens Examples ..................................... SUCCESS [7.455s]
> [INFO] Lens Ship Jars to Distributed Cache ............... SUCCESS [0.671s]
> [INFO] Lens Distribution ................................. SUCCESS [9.060s]
> [INFO] Lens ML Lib ....................................... SUCCESS [1:09.787s]
> [INFO] Lens ML Ext Distribution .......................... SUCCESS [2.321s]
> [INFO] Lens Regression ................................... SUCCESS [11.454s]
> [INFO] Lens UI ........................................... SUCCESS [27.047s]
> [INFO]
> ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO]
> ------------------------------------------------------------------------
> [INFO] Total time: 31:41.125s
> [INFO] Finished at: Thu May 19 11:53:17 UTC 2016
> [INFO] Final Memory: 186M/2279M
> [INFO]
> ------------------------------------------------------------------------
>
>
> Thanks,
>
> Archana H
>
>
