> On July 17, 2015, 2:52 p.m., Amareshwari Sriramadasu wrote: > > lens-server/src/main/java/org/apache/lens/server/metastore/JAXBUtils.java, > > line 814 > > <https://reviews.apache.org/r/36349/diff/4/?file=1013312#file1013312line814> > > > > No, I dont think we should clear. Partition can have its own internal > > properties set.
Then can you look at CubeMetastoreClient#getAllLatestPartsEquivalentTo? Whether clearing is fine there or not? - Rajat ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36349/#review92047 ----------------------------------------------------------- On July 16, 2015, 5:48 p.m., Rajat Khandelwal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36349/ > ----------------------------------------------------------- > > (Updated July 16, 2015, 5:48 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-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/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-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 > > Diff: https://reviews.apache.org/r/36349/diff/ > > > Testing > ------- > > [INFO] > ------------------------------------------------------------------------ > [INFO] Reactor Summary: > [INFO] > [INFO] Lens Checkstyle Rules ............................. SUCCESS [2.132s] > [INFO] Lens .............................................. SUCCESS [2.891s] > [INFO] Lens API .......................................... SUCCESS [20.190s] > [INFO] Lens API for server and extensions ................ SUCCESS [20.507s] > [INFO] Lens Cube ......................................... SUCCESS [4:32.908s] > [INFO] Lens DB storage ................................... SUCCESS [23.756s] > [INFO] Lens Query Library ................................ SUCCESS [16.700s] > [INFO] Lens Hive Driver .................................. SUCCESS [2:56.641s] > [INFO] Lens Driver for JDBC .............................. SUCCESS [36.084s] > [INFO] Lens Server ....................................... SUCCESS [5:22.969s] > [INFO] Lens client ....................................... SUCCESS [34.383s] > [INFO] Lens CLI .......................................... SUCCESS [2:37.563s] > [INFO] Lens Examples ..................................... SUCCESS [8.336s] > [INFO] Lens Distribution ................................. SUCCESS [8.085s] > [INFO] Lens ML Lib ....................................... SUCCESS [1:19.593s] > [INFO] Lens ML Ext Distribution .......................... SUCCESS [1.700s] > [INFO] Lens Regression ................................... SUCCESS [12.037s] > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 19:57.316s > [INFO] Finished at: Thu Jul 16 10:49:06 UTC 2015 > [INFO] Final Memory: 172M/1290M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Rajat Khandelwal > >
