----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33838/#review84865 -----------------------------------------------------------
lens-cli/src/test/java/org/apache/lens/cli/TestLensFactCommandsWithMissingWeight.java <https://reviews.apache.org/r/33838/#comment136281> Cube does not have weight. The name is misleading. Can we name cube_with_no_weight_facts or something else? lens-cli/src/test/java/org/apache/lens/cli/TestLensFactCommandsWithMissingWeight.java <https://reviews.apache.org/r/33838/#comment136280> Creation still not failing? lens-cli/src/test/java/org/apache/lens/cli/TestLensFactCommandsWithMissingWeight.java <https://reviews.apache.org/r/33838/#comment136283> If we are failing on exception and no fact is created - what was happening here ? lens-cli/src/test/resources/dim-local-storage-element.xml <https://reviews.apache.org/r/33838/#comment136284> Are there is changed required on example schema as well? Can you run sample-metastore and populate-metastore and make sure they are running fine? - Amareshwari Sriramadasu On May 21, 2015, 8:17 p.m., Yash Sharma wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/33838/ > ----------------------------------------------------------- > > (Updated May 21, 2015, 8:17 p.m.) > > > Review request for lens. > > > Repository: lens > > > Description > ------- > > NPE when weight is not set in XFactTable object while creating fact table > > > Diffs > ----- > > lens-api/src/main/resources/cube-0.1.xsd 1918e5c > lens-cli/src/test/java/org/apache/lens/cli/TestLensFactCommands.java > 19979cc > > lens-cli/src/test/java/org/apache/lens/cli/TestLensFactCommandsWithMissingWeight.java > PRE-CREATION > lens-cli/src/test/resources/dim-local-storage-element.xml 5a19cfa > lens-cli/src/test/resources/dim_table.xml 932c975 > lens-cli/src/test/resources/dim_table2.xml 1d6e138 > lens-cli/src/test/resources/fact-local-storage-element.xml 8ec3b42 > lens-cli/src/test/resources/fact_without_weight.xml PRE-CREATION > lens-cli/src/test/resources/sample-cube-without-wt.xml PRE-CREATION > lens-cli/src/test/resources/sample-cube.xml d54bcfb > lens-cli/src/test/resources/test-dimension.xml d602569 > lens-client/src/main/java/org/apache/lens/client/LensMetadataClient.java > a6651f6 > lens-examples/src/main/resources/dim_table2.xml eef1513 > lens-examples/src/main/resources/sample-cube.xml 0d97842 > > Diff: https://reviews.apache.org/r/33838/diff/ > > > Testing > ------- > > Yes. > TestCase: > mvn test -Dtest=TestLensFactCommandsWithMissingWeight#testFactCommands > > > Thanks, > > Yash Sharma > >
