> On May 19, 2015, 9:16 a.m., Amareshwari Sriramadasu wrote: > > src/site/apt/user/cli.apt, line 323 > > <https://reviews.apache.org/r/34348/diff/1/?file=962583#file962583line323> > > > > This move made prepQuery command to sit different from others. Can we > > put it along with other prepQuery commands as earlier ?
apparently `class.getMethods()` doesn't give methods in any particular order. So enforcing alphabetical ordering in commands now. - Rajat ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34348/#review84257 ----------------------------------------------------------- On May 18, 2015, 6:19 p.m., Rajat Khandelwal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34348/ > ----------------------------------------------------------- > > (Updated May 18, 2015, 6:19 p.m.) > > > Review request for lens. > > > Bugs: LENS-562 > https://issues.apache.org/jira/browse/LENS-562 > > > Repository: lens > > > Description > ------- > > made all changes. > > > Diffs > ----- > > lens-cli/src/main/java/org/apache/lens/cli/commands/LensCubeCommands.java > 1c83b54e2293b7dc5ee71efdb4093746820386d1 > lens-cli/src/main/java/org/apache/lens/cli/commands/LensFactCommands.java > 7ef7c23164dc8867383c1d22bd5b2b702d72629f > lens-cli/src/test/java/org/apache/lens/cli/TestLensFactCommands.java > 195bd435ae6eae6dd0b5988828156e7067d1b62a > lens-client/src/main/java/org/apache/lens/client/LensClient.java > 016e4ab1285582760b5da47b355402b747ea78e8 > lens-client/src/main/java/org/apache/lens/client/LensMetadataClient.java > 5e406b567664923ca34a24c58d77bc1781f0e4dd > > lens-cube/src/main/java/org/apache/lens/cube/metadata/CubeMetastoreClient.java > 08a63b8d92474e206a1d4099f9ac262e2cf91204 > > lens-cube/src/main/java/org/apache/lens/cube/metadata/timeline/PartitionTimeline.java > d27e43ee95b6188f135444cb2e0c9894a5e613b8 > lens-cube/src/test/java/org/apache/lens/cube/parse/CubeTestSetup.java > 1fe41737a906b9369b5cbafbfa303ab5dd777bb8 > > lens-server-api/src/main/java/org/apache/lens/server/api/metastore/CubeMetastoreService.java > cde85f6c1262b1efa76839946bfd1c0f218a03c9 > > lens-server/src/main/java/org/apache/lens/server/metastore/CubeMetastoreServiceImpl.java > b0f61b6894770717b7ffb356fa8b9e8bcf637b19 > > lens-server/src/main/java/org/apache/lens/server/metastore/MetastoreResource.java > 73e8750aa45e9e4407d4b0e24fa6b335f5798e45 > lens-server/src/test/java/org/apache/lens/server/LensTestUtil.java > 859e9ccc947e7f06a381d2db612f676580699d12 > src/site/apt/user/cli.apt bd394972790792905cb21dd0da5894840f404d8e > > Diff: https://reviews.apache.org/r/34348/diff/ > > > Testing > ------- > > unit and integration tests added > unit and integration tests added > > > Thanks, > > Rajat Khandelwal > >
