You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@lens.apache.org by Amareshwari Sriramadasu <am...@apache.org> on 2015/03/30 12:16:24 UTC

Review Request 32615: Remove counts and weights from QueryPlan and DriverQueryPlan

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

Review request for lens, Jaideep dhok and Rajat Khandelwal.


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


Repository: lens


Description
-------

Simply removes counts and weights from QueryPlan and DriverQueryPlan


Diffs
-----

  lens-api/src/main/java/org/apache/lens/api/query/QueryPlan.java a578759 
  lens-cube/src/main/java/org/apache/lens/driver/cube/RewriterPlan.java b05814e 
  lens-cube/src/test/java/org/apache/lens/cube/parse/TestRewriterPlan.java 0738b27 
  lens-driver-hive/src/main/java/org/apache/lens/driver/hive/HiveQueryPlan.java fce93a9 
  lens-driver-hive/src/test/java/org/apache/lens/driver/hive/TestHiveDriver.java b16c346 
  lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryPlan.java 1d41720 
  lens-server/src/test/java/org/apache/lens/server/query/TestQueryService.java e3e3d4b 

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


Testing
-------

[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO] 
[INFO] Lens Checkstyle Rules ............................. SUCCESS [2.520s]
[INFO] Lens .............................................. SUCCESS [2.450s]
[INFO] Lens API .......................................... SUCCESS [9.564s]
[INFO] Lens API for server and extensions ................ SUCCESS [8.661s]
[INFO] Lens Cube ......................................... SUCCESS [2:49.387s]
[INFO] Lens DB storage ................................... SUCCESS [13.161s]
[INFO] Lens Query Library ................................ SUCCESS [7.701s]
[INFO] Lens Hive Driver .................................. SUCCESS [3:06.058s]
[INFO] Lens Driver for JDBC .............................. SUCCESS [20.838s]
[INFO] Lens Server ....................................... SUCCESS [7:02.908s]
[INFO] Lens client ....................................... SUCCESS [27.563s]
[INFO] Lens CLI .......................................... SUCCESS [2:23.637s]
[INFO] Lens Examples ..................................... SUCCESS [0.895s]
[INFO] Lens Distribution ................................. SUCCESS [9.316s]
[INFO] Lens ML Lib ....................................... SUCCESS [1:05.101s]
[INFO] Lens ML Ext Distribution .......................... SUCCESS [2.656s]
[INFO] Lens Regression ................................... SUCCESS [0.489s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 18:13.986s
[INFO] Finished at: Mon Mar 30 10:15:39 UTC 2015
[INFO] Final Memory: 108M/1077M
[INFO] ------------------------------------------------------------------------


Thanks,

Amareshwari Sriramadasu


Re: Review Request 32615: LENS-437 :Remove counts and weights from QueryPlan and DriverQueryPlan

Posted by Jaideep dhok <ja...@inmobi.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32615/#review78196
-----------------------------------------------------------

Ship it!


Ship It!

- Jaideep dhok


On March 30, 2015, 10:27 a.m., Amareshwari Sriramadasu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/32615/
> -----------------------------------------------------------
> 
> (Updated March 30, 2015, 10:27 a.m.)
> 
> 
> Review request for lens, Jaideep dhok and Rajat Khandelwal.
> 
> 
> Bugs: LENS-437
>     https://issues.apache.org/jira/browse/LENS-437
> 
> 
> Repository: lens
> 
> 
> Description
> -------
> 
> Simply removes counts and weights from QueryPlan and DriverQueryPlan
> 
> 
> Diffs
> -----
> 
>   lens-api/src/main/java/org/apache/lens/api/query/QueryPlan.java a578759 
>   lens-cube/src/main/java/org/apache/lens/driver/cube/RewriterPlan.java b05814e 
>   lens-cube/src/test/java/org/apache/lens/cube/parse/TestRewriterPlan.java 0738b27 
>   lens-driver-hive/src/main/java/org/apache/lens/driver/hive/HiveQueryPlan.java fce93a9 
>   lens-driver-hive/src/test/java/org/apache/lens/driver/hive/TestHiveDriver.java b16c346 
>   lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryPlan.java 1d41720 
>   lens-server/src/test/java/org/apache/lens/server/query/TestQueryService.java e3e3d4b 
> 
> Diff: https://reviews.apache.org/r/32615/diff/
> 
> 
> Testing
> -------
> 
> [INFO] ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO] 
> [INFO] Lens Checkstyle Rules ............................. SUCCESS [2.520s]
> [INFO] Lens .............................................. SUCCESS [2.450s]
> [INFO] Lens API .......................................... SUCCESS [9.564s]
> [INFO] Lens API for server and extensions ................ SUCCESS [8.661s]
> [INFO] Lens Cube ......................................... SUCCESS [2:49.387s]
> [INFO] Lens DB storage ................................... SUCCESS [13.161s]
> [INFO] Lens Query Library ................................ SUCCESS [7.701s]
> [INFO] Lens Hive Driver .................................. SUCCESS [3:06.058s]
> [INFO] Lens Driver for JDBC .............................. SUCCESS [20.838s]
> [INFO] Lens Server ....................................... SUCCESS [7:02.908s]
> [INFO] Lens client ....................................... SUCCESS [27.563s]
> [INFO] Lens CLI .......................................... SUCCESS [2:23.637s]
> [INFO] Lens Examples ..................................... SUCCESS [0.895s]
> [INFO] Lens Distribution ................................. SUCCESS [9.316s]
> [INFO] Lens ML Lib ....................................... SUCCESS [1:05.101s]
> [INFO] Lens ML Ext Distribution .......................... SUCCESS [2.656s]
> [INFO] Lens Regression ................................... SUCCESS [0.489s]
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 18:13.986s
> [INFO] Finished at: Mon Mar 30 10:15:39 UTC 2015
> [INFO] Final Memory: 108M/1077M
> [INFO] ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Amareshwari Sriramadasu
> 
>


Re: Review Request 32615: LENS-437 :Remove counts and weights from QueryPlan and DriverQueryPlan

Posted by Amareshwari Sriramadasu <am...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32615/
-----------------------------------------------------------

(Updated March 30, 2015, 10:27 a.m.)


Review request for lens, Jaideep dhok and Rajat Khandelwal.


Summary (updated)
-----------------

LENS-437 :Remove counts and weights from QueryPlan and DriverQueryPlan


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


Repository: lens


Description
-------

Simply removes counts and weights from QueryPlan and DriverQueryPlan


Diffs
-----

  lens-api/src/main/java/org/apache/lens/api/query/QueryPlan.java a578759 
  lens-cube/src/main/java/org/apache/lens/driver/cube/RewriterPlan.java b05814e 
  lens-cube/src/test/java/org/apache/lens/cube/parse/TestRewriterPlan.java 0738b27 
  lens-driver-hive/src/main/java/org/apache/lens/driver/hive/HiveQueryPlan.java fce93a9 
  lens-driver-hive/src/test/java/org/apache/lens/driver/hive/TestHiveDriver.java b16c346 
  lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryPlan.java 1d41720 
  lens-server/src/test/java/org/apache/lens/server/query/TestQueryService.java e3e3d4b 

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


Testing
-------

[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO] 
[INFO] Lens Checkstyle Rules ............................. SUCCESS [2.520s]
[INFO] Lens .............................................. SUCCESS [2.450s]
[INFO] Lens API .......................................... SUCCESS [9.564s]
[INFO] Lens API for server and extensions ................ SUCCESS [8.661s]
[INFO] Lens Cube ......................................... SUCCESS [2:49.387s]
[INFO] Lens DB storage ................................... SUCCESS [13.161s]
[INFO] Lens Query Library ................................ SUCCESS [7.701s]
[INFO] Lens Hive Driver .................................. SUCCESS [3:06.058s]
[INFO] Lens Driver for JDBC .............................. SUCCESS [20.838s]
[INFO] Lens Server ....................................... SUCCESS [7:02.908s]
[INFO] Lens client ....................................... SUCCESS [27.563s]
[INFO] Lens CLI .......................................... SUCCESS [2:23.637s]
[INFO] Lens Examples ..................................... SUCCESS [0.895s]
[INFO] Lens Distribution ................................. SUCCESS [9.316s]
[INFO] Lens ML Lib ....................................... SUCCESS [1:05.101s]
[INFO] Lens ML Ext Distribution .......................... SUCCESS [2.656s]
[INFO] Lens Regression ................................... SUCCESS [0.489s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 18:13.986s
[INFO] Finished at: Mon Mar 30 10:15:39 UTC 2015
[INFO] Final Memory: 108M/1077M
[INFO] ------------------------------------------------------------------------


Thanks,

Amareshwari Sriramadasu