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

Ship it!


Ship It!

- Jaideep dhok


On March 4, 2015, 6:37 p.m., Amareshwari Sriramadasu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/31733/
> -----------------------------------------------------------
> 
> (Updated March 4, 2015, 6:37 p.m.)
> 
> 
> Review request for lens and Rajat Khandelwal.
> 
> 
> Bugs: LENS-296
>     https://issues.apache.org/jira/browse/LENS-296
> 
> 
> Repository: lens
> 
> 
> Description
> -------
> 
> Was able to reproduce NPE reported in the jira with testcase added. NPE was 
> happening because of addQueriedTable() return not handled in resolvers.
> 
> 
> Diffs
> -----
> 
>   lens-cube/src/main/java/org/apache/lens/cube/parse/ColumnResolver.java 
> 97cfeb2 
>   lens-cube/src/main/java/org/apache/lens/cube/parse/ExpressionResolver.java 
> ead0b89 
>   
> lens-cube/src/test/java/org/apache/lens/cube/parse/TestDenormalizationResolver.java
>  0d7db53 
> 
> Diff: https://reviews.apache.org/r/31733/diff/
> 
> 
> Testing
> -------
> 
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO] 
> [INFO] Lens Checkstyle Rules ............................. SUCCESS [2.739s]
> [INFO] Lens .............................................. SUCCESS [2.592s]
> [INFO] Lens API .......................................... SUCCESS [11.223s]
> [INFO] Lens API for server and extensions ................ SUCCESS [6.629s]
> [INFO] Lens Cube ......................................... SUCCESS [7:25.447s]
> [INFO] Lens DB storage ................................... SUCCESS [9.657s]
> [INFO] Lens Query Library ................................ SUCCESS [4.887s]
> [INFO] Lens Hive Driver .................................. SUCCESS [2:37.392s]
> [INFO] Lens Driver for JDBC .............................. SUCCESS [25.923s]
> [INFO] Lens Server ....................................... SUCCESS [4:33.828s]
> [INFO] Lens client ....................................... SUCCESS [20.521s]
> [INFO] Lens CLI .......................................... SUCCESS [1:39.595s]
> [INFO] Lens Examples ..................................... SUCCESS [0.790s]
> [INFO] Lens Distribution ................................. SUCCESS [9.319s]
> [INFO] Lens ML Lib ....................................... SUCCESS [43.041s]
> [INFO] Lens Regression ................................... SUCCESS [0.436s]
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] Total time: 18:35.436s
> [INFO] Finished at: Wed Mar 04 18:36:14 UTC 2015
> [INFO] Final Memory: 110M/1033M
> [INFO] 
> ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Amareshwari Sriramadasu
> 
>

Reply via email to