> On July 20, 2015, 3:45 p.m., Amareshwari Sriramadasu wrote:
> > lens-server/src/main/java/org/apache/lens/server/metastore/JAXBUtils.java, 
> > lines 819-823
> > <https://reviews.apache.org/r/36349/diff/6/?file=1016537#file1016537line819>
> >
> >     Do we have tests around parsable and unparsable part spec code flow?

They do now. full spec is sent always and time + non time are sent additionally 
if possible.


- Rajat


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


On July 20, 2015, 5:37 p.m., Rajat Khandelwal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/36349/
> -----------------------------------------------------------
> 
> (Updated July 20, 2015, 5:37 p.m.)
> 
> 
> Review request for lens.
> 
> 
> Bugs: LENS-652
>     https://issues.apache.org/jira/browse/LENS-652
> 
> 
> Repository: lens
> 
> 
> Description
> -------
> 
> Our Requirement is to be able to change path of an existing partition.
> 
> 
> Diffs
> -----
> 
>   lens-api/pom.xml 507dd01a92671fe5b2be3f0a5156b24817691d7d 
>   lens-cli/src/main/java/org/apache/lens/cli/commands/BaseLensCommand.java 
> 74b4e91e894ddc4995e2ddd36152ab97204294c0 
>   
> lens-cli/src/main/java/org/apache/lens/cli/commands/LensDimensionTableCommands.java
>  6a9339365b7980d1db7f9c0f5bb8c414956611f8 
>   lens-cli/src/main/java/org/apache/lens/cli/commands/LensFactCommands.java 
> bdb9c382fc0badfa19604aeeb329953b8919dc0b 
>   
> lens-cli/src/main/java/org/apache/lens/cli/commands/PhysicalTableCrudCommand.java
>  a479c14d37f39b0a5fd95801a5d6b69e3f2037ff 
>   lens-cli/src/test/java/org/apache/lens/cli/LensCliApplicationTest.java 
> 2de0cad507597e5c0dd58cd356e74dac6e3bca33 
>   
> lens-cli/src/test/java/org/apache/lens/cli/TestLensDimensionTableCommands.java
>  a87d0b7a1d5d4a89108e8a9c5718174b0e143531 
>   lens-cli/src/test/java/org/apache/lens/cli/TestLensFactCommands.java 
> 13bfbd2469042cdb8751d54a670d4904af48c065 
>   lens-cli/src/test/resources/dim1-local-parts.xml 
> 87fbc6da94a1a7dfefb3474644bbd2992771e6ee 
>   lens-cli/src/test/resources/fact1-local-parts.xml 
> 6201320787d481a5ef87459dad59ace1812e2abb 
>   lens-client/src/main/java/org/apache/lens/client/LensClient.java 
> b15b024369b8b8b1229f8923f0c809936f10577f 
>   lens-client/src/main/java/org/apache/lens/client/LensMetadataClient.java 
> b4e05524debaa747674014607a6047b9c3b3b63b 
>   
> lens-cube/src/main/java/org/apache/lens/cube/metadata/CubeMetastoreClient.java
>  dc2d9f64e5c9ea2588e5a58814a82d75854bfdd7 
>   lens-cube/src/main/java/org/apache/lens/cube/metadata/HDFSStorage.java 
> 8107f4c2f656a94246bf89f1d6e159bbeb9f32d3 
>   lens-cube/src/main/java/org/apache/lens/cube/metadata/MetastoreUtil.java 
> 8e2c9ed23670934a76d931ad6eb018bcf203dedf 
>   
> lens-cube/src/main/java/org/apache/lens/cube/metadata/PartitionMetahook.java 
> 1174d254a54539be7479bfa93363e65061bcd7c3 
>   lens-cube/src/main/java/org/apache/lens/cube/metadata/Storage.java 
> c7540420c19f54fdfe5139cecb581f1417351a4d 
>   lens-cube/src/main/java/org/apache/lens/cube/metadata/TimePartition.java 
> 7c966d8e61e04572ed2f3b851862b86b2d082612 
>   lens-cube/src/main/java/org/apache/lens/cube/metadata/UpdatePeriod.java 
> 966dbdf606ff379c98af06b8c733b2be4d3ce692 
>   
> lens-cube/src/main/java/org/apache/lens/cube/parse/StorageTableResolver.java 
> ce6f43488a34f945842c7560f159870322136776 
>   
> lens-cube/src/test/java/org/apache/lens/cube/metadata/TestCubeMetastoreClient.java
>  18b6c876abdf925b1c718b606377b72f90c4466b 
>   lens-cube/src/test/java/org/apache/lens/cube/parse/CubeTestSetup.java 
> f9398588fc3127708fe57960771c9b02296e93ab 
>   lens-driver-hive/src/main/resources/hivedriver-default.xml 
> 85a3f89deba44a9ca5ef4ffd41074b1405d846b6 
>   
> lens-server-api/src/main/java/org/apache/lens/server/api/metastore/CubeMetastoreService.java
>  6226521545fc1dc478c41595c58a97e2f800f47a 
>   
> lens-server/src/main/java/org/apache/lens/server/metastore/CubeMetastoreServiceImpl.java
>  c5e1c14377b9ffe54e69a841559ab73df61ca162 
>   lens-server/src/main/java/org/apache/lens/server/metastore/JAXBUtils.java 
> 811fd329141bb6f192c11df0bbde8fd4a89bbd3c 
>   
> lens-server/src/main/java/org/apache/lens/server/metastore/MetastoreResource.java
>  bfe4813b78ac98d45e0635d69c40ab6c5ae30fa0 
>   
> lens-server/src/test/java/org/apache/lens/server/metastore/TestMetastoreService.java
>  dc1dc1d52a78cfc2e11ece5a0ef9b2c2ce251a7a 
>   lens-storage-db/src/main/java/org/apache/lens/storage/db/DBStorage.java 
> e6d413252c1b74770e4f1cadd8ffa0423b400f11 
>   lens-storage-db/src/test/java/org/apache/lens/storage/db/TestDBStorage.java 
> 1e2482809986d745e7767661d414d8d99a685142 
>   src/site/apt/admin/hivedriver-config.apt 
> b668e160f4fd1ce05179bb50c7dc6264ea6101e2 
>   src/site/apt/user/cli.apt 353c1717a94d8899e0e66afa69c8db4e78b46a3d 
> 
> Diff: https://reviews.apache.org/r/36349/diff/
> 
> 
> Testing
> -------
> 
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO] 
> [INFO] Lens Checkstyle Rules ............................. SUCCESS [2.054s]
> [INFO] Lens .............................................. SUCCESS [3.398s]
> [INFO] Lens API .......................................... SUCCESS [21.952s]
> [INFO] Lens API for server and extensions ................ SUCCESS [20.166s]
> [INFO] Lens Cube ......................................... SUCCESS [4:08.672s]
> [INFO] Lens DB storage ................................... SUCCESS [18.911s]
> [INFO] Lens Query Library ................................ SUCCESS [16.230s]
> [INFO] Lens Hive Driver .................................. SUCCESS [2:47.532s]
> [INFO] Lens Driver for JDBC .............................. SUCCESS [34.907s]
> [INFO] Lens Server ....................................... SUCCESS [5:27.831s]
> [INFO] Lens client ....................................... SUCCESS [35.633s]
> [INFO] Lens CLI .......................................... SUCCESS [2:47.536s]
> [INFO] Lens Examples ..................................... SUCCESS [9.378s]
> [INFO] Lens Distribution ................................. SUCCESS [11.749s]
> [INFO] Lens ML Lib ....................................... SUCCESS [1:25.497s]
> [INFO] Lens ML Ext Distribution .......................... SUCCESS [2.054s]
> [INFO] Lens Regression ................................... SUCCESS [11.638s]
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] Total time: 19:45.976s
> [INFO] Finished at: Mon Jul 20 11:10:37 UTC 2015
> [INFO] Final Memory: 193M/1771M
> [INFO] 
> ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Rajat Khandelwal
> 
>

Reply via email to