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

(Updated April 19, 2016, 8:36 a.m.)


Review request for lens.


Changes
-------

Addressed review comments.


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


Repository: lens


Description
-------

Tags are key value pairs associated with a cube column eg : category, 
visible-on-ui and etc.


Diffs (updated)
-----

  lens-api/src/main/resources/cube-0.1.xsd f221a6d 
  lens-cli/src/test/java/org/apache/lens/cli/TestLensCubeCommands.java 5a353df 
  lens-cli/src/test/resources/sample-cube.xml 9bcf177 
  lens-cube/src/main/java/org/apache/lens/cube/metadata/BaseDimAttribute.java 
bd4ae57 
  lens-cube/src/main/java/org/apache/lens/cube/metadata/ColumnMeasure.java 
5fda721 
  lens-cube/src/main/java/org/apache/lens/cube/metadata/CubeColumn.java 77024c0 
  lens-cube/src/main/java/org/apache/lens/cube/metadata/CubeDimAttribute.java 
26c24de 
  lens-cube/src/main/java/org/apache/lens/cube/metadata/CubeMeasure.java 
d5fc0e7 
  lens-cube/src/main/java/org/apache/lens/cube/metadata/ExprColumn.java da87e31 
  lens-cube/src/main/java/org/apache/lens/cube/metadata/MetastoreConstants.java 
cd4862c 
  lens-cube/src/main/java/org/apache/lens/cube/metadata/MetastoreUtil.java 
399d5a7 
  
lens-cube/src/main/java/org/apache/lens/cube/metadata/ReferencedDimAttribute.java
 9a1c44b 
  
lens-cube/src/test/java/org/apache/lens/cube/metadata/TestCubeMetastoreClient.java
 18575f6 
  lens-examples/src/main/resources/sample-cube.xml 4046e82 
  
lens-examples/src/test/java/org/apache/lens/examples/ExampleSchemaToStringTest.java
 b884368 
  lens-examples/src/test/resources/yaml/sales-cube.yaml 935e3f6 
  lens-examples/src/test/resources/yaml/sample-cube.yaml 3f2ffc0 
  lens-server/src/main/java/org/apache/lens/server/metastore/JAXBUtils.java 
6571dae 

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


Testing (updated)
-------

Added new unit test and updated existing ones. mvn clean install runs fine.

[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO] 
[INFO] Lens Checkstyle Rules ............................. SUCCESS [2.291s]
[INFO] Lens .............................................. SUCCESS [5.163s]
[INFO] Lens API .......................................... SUCCESS [31.410s]
[INFO] Lens API for server and extensions ................ SUCCESS [21.962s]
[INFO] Lens Cube ......................................... SUCCESS [18:30.453s]
[INFO] Lens DB storage ................................... SUCCESS [19.615s]
[INFO] Lens Query Library ................................ SUCCESS [16.384s]
[INFO] Lens Hive Driver .................................. SUCCESS [3:13.159s]
[INFO] Lens Driver for JDBC .............................. SUCCESS [36.141s]
[INFO] Lens Elastic Search Driver ........................ SUCCESS [18.043s]
[INFO] Lens Server ....................................... SUCCESS [17:01.442s]
[INFO] Lens client ....................................... SUCCESS [36.402s]
[INFO] Lens CLI .......................................... SUCCESS [3:14.367s]
[INFO] Lens Examples ..................................... SUCCESS [12.015s]
[INFO] Lens Ship Jars to Distributed Cache ............... SUCCESS [0.770s]
[INFO] Lens Distribution ................................. SUCCESS [10.500s]
[INFO] Lens ML Lib ....................................... SUCCESS [1:24.982s]
[INFO] Lens ML Ext Distribution .......................... SUCCESS [1.996s]
[INFO] Lens Regression ................................... SUCCESS [15.048s]
[INFO] Lens UI ........................................... SUCCESS [50.544s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 48:03.576s
[INFO] Finished at: Tue Apr 19 08:20:15 UTC 2016
[INFO] Final Memory: 172M/2793M
[INFO] ------------------------------------------------------------------------


Thanks,

Sushil Mohanty

Reply via email to