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/20 06:40:27 UTC

Review Request 32280: LENS-420 : Fix Metric gauge creation in RewriteUtil for multiple cube query

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

Review request for lens and Jaideep dhok.


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


Repository: lens


Description
-------

Fixes the gauge creation to have unique name for all queies


Diffs
-----

  lens-cube/src/main/java/org/apache/lens/driver/cube/RewriteUtil.java 3dc66ac 
  lens-cube/src/test/java/org/apache/lens/driver/cube/TestRewriting.java 34d7dca 
  lens-server/src/test/java/org/apache/lens/server/query/TestQueryService.java 4c84c8b 

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


Testing
-------

[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO] 
[INFO] Lens Checkstyle Rules ............................. SUCCESS [2.160s]
[INFO] Lens .............................................. SUCCESS [1.797s]
[INFO] Lens API .......................................... SUCCESS [4.892s]
[INFO] Lens API for server and extensions ................ SUCCESS [6.314s]
[INFO] Lens Cube ......................................... SUCCESS [2:04.395s]
[INFO] Lens DB storage ................................... SUCCESS [10.329s]
[INFO] Lens Query Library ................................ SUCCESS [4.413s]
[INFO] Lens Hive Driver .................................. SUCCESS [2:36.381s]
[INFO] Lens Driver for JDBC .............................. SUCCESS [18.258s]
[INFO] Lens Server ....................................... SUCCESS [4:24.649s]
[INFO] Lens client ....................................... SUCCESS [20.163s]
[INFO] Lens CLI .......................................... SUCCESS [2:52.956s]
[INFO] Lens Examples ..................................... SUCCESS [0.784s]
[INFO] Lens Distribution ................................. SUCCESS [8.962s]
[INFO] Lens ML Lib ....................................... SUCCESS [42.906s]
[INFO] Lens Regression ................................... SUCCESS [0.485s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 14:00.856s
[INFO] Finished at: Fri Mar 20 05:37:34 UTC 2015
[INFO] Final Memory: 100M/1241M
[INFO] ------------------------------------------------------------------------


Thanks,

Amareshwari Sriramadasu


Re: Review Request 32280: LENS-420 : Fix Metric gauge creation in RewriteUtil for multiple cube query

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

Ship it!


Ship It!

- Jaideep dhok


On March 20, 2015, 5:40 a.m., Amareshwari Sriramadasu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/32280/
> -----------------------------------------------------------
> 
> (Updated March 20, 2015, 5:40 a.m.)
> 
> 
> Review request for lens and Jaideep dhok.
> 
> 
> Bugs: LENS-420
>     https://issues.apache.org/jira/browse/LENS-420
> 
> 
> Repository: lens
> 
> 
> Description
> -------
> 
> Fixes the gauge creation to have unique name for all queies
> 
> 
> Diffs
> -----
> 
>   lens-cube/src/main/java/org/apache/lens/driver/cube/RewriteUtil.java 3dc66ac 
>   lens-cube/src/test/java/org/apache/lens/driver/cube/TestRewriting.java 34d7dca 
>   lens-server/src/test/java/org/apache/lens/server/query/TestQueryService.java 4c84c8b 
> 
> Diff: https://reviews.apache.org/r/32280/diff/
> 
> 
> Testing
> -------
> 
> [INFO] ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO] 
> [INFO] Lens Checkstyle Rules ............................. SUCCESS [2.160s]
> [INFO] Lens .............................................. SUCCESS [1.797s]
> [INFO] Lens API .......................................... SUCCESS [4.892s]
> [INFO] Lens API for server and extensions ................ SUCCESS [6.314s]
> [INFO] Lens Cube ......................................... SUCCESS [2:04.395s]
> [INFO] Lens DB storage ................................... SUCCESS [10.329s]
> [INFO] Lens Query Library ................................ SUCCESS [4.413s]
> [INFO] Lens Hive Driver .................................. SUCCESS [2:36.381s]
> [INFO] Lens Driver for JDBC .............................. SUCCESS [18.258s]
> [INFO] Lens Server ....................................... SUCCESS [4:24.649s]
> [INFO] Lens client ....................................... SUCCESS [20.163s]
> [INFO] Lens CLI .......................................... SUCCESS [2:52.956s]
> [INFO] Lens Examples ..................................... SUCCESS [0.784s]
> [INFO] Lens Distribution ................................. SUCCESS [8.962s]
> [INFO] Lens ML Lib ....................................... SUCCESS [42.906s]
> [INFO] Lens Regression ................................... SUCCESS [0.485s]
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 14:00.856s
> [INFO] Finished at: Fri Mar 20 05:37:34 UTC 2015
> [INFO] Final Memory: 100M/1241M
> [INFO] ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Amareshwari Sriramadasu
> 
>