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

(Updated April 12, 2017, 12:10 p.m.)


Review request for lens.


Changes
-------

- Fixed all checkstyle issue.
- Fixed failing lens-examples test cases


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


Repository: lens


Description
-------

This is a neew feature to support union across multiple facts.


Diffs (updated)
-----

  lens-api/src/main/java/org/apache/lens/api/ToXMLString.java e74adc9 
  lens-api/src/main/java/org/apache/lens/api/jaxb/LensJAXBContext.java 14fc4aa 
  lens-api/src/main/java/org/apache/lens/api/metastore/SchemaTraverser.java 
PRE-CREATION 
  lens-api/src/main/resources/cube-0.1.xsd 060eb43 
  lens-api/src/main/resources/lens-errors.conf 29e24cf 
  lens-cli/src/main/java/org/apache/lens/cli/commands/LensSchemaCommands.java 
feabf9c 
  lens-cube/src/main/java/org/apache/lens/cube/error/LensCubeErrorCode.java 
571b481 
  
lens-cube/src/main/java/org/apache/lens/cube/error/NoCandidateFactAvailableException.java
 b2568ff 
  lens-cube/src/main/java/org/apache/lens/cube/metadata/CubeFactTable.java 
896a7a1 
  
lens-cube/src/main/java/org/apache/lens/cube/metadata/CubeMetastoreClient.java 
087c203 
  lens-cube/src/main/java/org/apache/lens/cube/metadata/DateUtil.java 7717081 
  lens-cube/src/main/java/org/apache/lens/cube/metadata/FactPartition.java 
1694b80 
  lens-cube/src/main/java/org/apache/lens/cube/metadata/JAXBUtils.java 
PRE-CREATION 
  lens-cube/src/main/java/org/apache/lens/cube/metadata/MetastoreUtil.java 
4e350c8 
  lens-cube/src/main/java/org/apache/lens/cube/metadata/TimeRange.java bf6cc5c 
  
lens-cube/src/main/java/org/apache/lens/cube/parse/AbridgedTimeRangeWriter.java 
8681e90 
  lens-cube/src/main/java/org/apache/lens/cube/parse/AggregateResolver.java 
9658100 
  
lens-cube/src/main/java/org/apache/lens/cube/parse/BetweenTimeRangeWriter.java 
c8b8129 
  lens-cube/src/main/java/org/apache/lens/cube/parse/Candidate.java 
PRE-CREATION 
  
lens-cube/src/main/java/org/apache/lens/cube/parse/CandidateCoveringSetsResolver.java
 PRE-CREATION 
  lens-cube/src/main/java/org/apache/lens/cube/parse/CandidateDim.java 4dcdbcf 
  lens-cube/src/main/java/org/apache/lens/cube/parse/CandidateFact.java b42262d 
  lens-cube/src/main/java/org/apache/lens/cube/parse/CandidateTable.java 
e001ca4 
  
lens-cube/src/main/java/org/apache/lens/cube/parse/CandidateTablePruneCause.java
 bd6e27c 
  
lens-cube/src/main/java/org/apache/lens/cube/parse/CandidateTableResolver.java 
ed37bc5 
  lens-cube/src/main/java/org/apache/lens/cube/parse/CandidateUtil.java 
PRE-CREATION 
  lens-cube/src/main/java/org/apache/lens/cube/parse/CheckTableNames.java 
8586262 
  lens-cube/src/main/java/org/apache/lens/cube/parse/ColumnLifetimeChecker.java 
PRE-CREATION 
  lens-cube/src/main/java/org/apache/lens/cube/parse/ColumnResolver.java 
4d8910a 
  lens-cube/src/main/java/org/apache/lens/cube/parse/CubeQueryContext.java 
125b432 
  lens-cube/src/main/java/org/apache/lens/cube/parse/CubeQueryRewriter.java 
ea2eb7e 
  lens-cube/src/main/java/org/apache/lens/cube/parse/CubeSemanticAnalyzer.java 
fc96055 
  lens-cube/src/main/java/org/apache/lens/cube/parse/DefaultAliasDecider.java 
80ceae4 
  lens-cube/src/main/java/org/apache/lens/cube/parse/DefaultQueryAST.java 
c9993f3 
  
lens-cube/src/main/java/org/apache/lens/cube/parse/DenormalizationResolver.java 
cb26878 
  lens-cube/src/main/java/org/apache/lens/cube/parse/ExpressionResolver.java 
c99fdf1 
  lens-cube/src/main/java/org/apache/lens/cube/parse/FieldValidator.java 
48af0c9 
  lens-cube/src/main/java/org/apache/lens/cube/parse/GroupbyResolver.java 
052b87a 
  lens-cube/src/main/java/org/apache/lens/cube/parse/JoinCandidate.java 
PRE-CREATION 
  lens-cube/src/main/java/org/apache/lens/cube/parse/JoinResolver.java fce1662 
  
lens-cube/src/main/java/org/apache/lens/cube/parse/LeastPartitionResolver.java 
0bc7f82 
  lens-cube/src/main/java/org/apache/lens/cube/parse/LightestFactResolver.java 
4356401 
  
lens-cube/src/main/java/org/apache/lens/cube/parse/MaxCoveringFactResolver.java 
2822857 
  lens-cube/src/main/java/org/apache/lens/cube/parse/MultiFactHQLContext.java 
979c24b 
  lens-cube/src/main/java/org/apache/lens/cube/parse/PruneCauses.java 9b5a52f 
  lens-cube/src/main/java/org/apache/lens/cube/parse/QueriedPhraseContext.java 
34a562d 
  lens-cube/src/main/java/org/apache/lens/cube/parse/QueryAST.java 7298604 
  lens-cube/src/main/java/org/apache/lens/cube/parse/SimpleHQLContext.java 
62ceb12 
  
lens-cube/src/main/java/org/apache/lens/cube/parse/SingleFactMultiStorageHQLContext.java
 9b48213 
  
lens-cube/src/main/java/org/apache/lens/cube/parse/SingleFactSingleStorageHQLContext.java
 dbc84ed 
  lens-cube/src/main/java/org/apache/lens/cube/parse/StorageCandidate.java 
PRE-CREATION 
  lens-cube/src/main/java/org/apache/lens/cube/parse/StorageTableResolver.java 
2b63193 
  lens-cube/src/main/java/org/apache/lens/cube/parse/StorageUtil.java f9636d1 
  lens-cube/src/main/java/org/apache/lens/cube/parse/TimeRangeChecker.java 
f18ae36 
  lens-cube/src/main/java/org/apache/lens/cube/parse/UnionCandidate.java 
PRE-CREATION 
  lens-cube/src/main/java/org/apache/lens/cube/parse/UnionHQLContext.java 
e6ee989 
  lens-cube/src/main/java/org/apache/lens/cube/parse/UnionQueryWriter.java 
PRE-CREATION 
  lens-cube/src/main/java/org/apache/lens/cube/parse/join/AutoJoinContext.java 
3d5c5ac 
  
lens-cube/src/main/java/org/apache/lens/cube/parse/join/BridgeTableJoinContext.java
 cf74634 
  lens-cube/src/main/java/org/apache/lens/driver/cube/RewriterPlan.java fd6c30d 
  lens-cube/src/test/java/org/apache/lens/cube/metadata/DateFactory.java 
3bdc047 
  lens-cube/src/test/java/org/apache/lens/cube/parse/CubeTestSetup.java 9b29083 
  
lens-cube/src/test/java/org/apache/lens/cube/parse/FieldsCannotBeQueriedTogetherTest.java
 1a5bd0d 
  lens-cube/src/test/java/org/apache/lens/cube/parse/TestAggregateResolver.java 
dd0b6dc 
  lens-cube/src/test/java/org/apache/lens/cube/parse/TestBaseCubeQueries.java 
dbb8fa3 
  
lens-cube/src/test/java/org/apache/lens/cube/parse/TestBetweenTimeRangeWriter.java
 cabb95e 
  
lens-cube/src/test/java/org/apache/lens/cube/parse/TestBridgeTableQueries.java 
2f00244 
  lens-cube/src/test/java/org/apache/lens/cube/parse/TestCubeRewriter.java 
7e06a5c 
  
lens-cube/src/test/java/org/apache/lens/cube/parse/TestDenormalizationResolver.java
 5505ed4 
  
lens-cube/src/test/java/org/apache/lens/cube/parse/TestExpressionResolver.java 
5d4e87f 
  lens-cube/src/test/java/org/apache/lens/cube/parse/TestJoinResolver.java 
677d641 
  lens-cube/src/test/java/org/apache/lens/cube/parse/TestQueryMetrics.java 
27a18f4 
  lens-cube/src/test/java/org/apache/lens/cube/parse/TestRewriterPlan.java 
7f26b24 
  lens-cube/src/test/java/org/apache/lens/cube/parse/TestTimeRangeResolver.java 
280a8c4 
  
lens-cube/src/test/java/org/apache/lens/cube/parse/TestTimeRangeWriterWithQuery.java
 7010849 
  
lens-cube/src/test/java/org/apache/lens/cube/parse/TestUnionAndJoinCandidates.java
 PRE-CREATION 
  lens-cube/src/test/java/org/apache/lens/cube/parse/TestUnionQueries.java 
d5bc81c 
  lens-cube/src/test/resources/schema/cubes/base/basecube.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/cubes/base/testcube.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/cubes/derived/der1.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/cubes/derived/der2.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/cubes/derived/der3.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/cubes/derived/derivedcube.xml 
PRE-CREATION 
  lens-cube/src/test/resources/schema/cubes/derived/union_join_ctx_der1.xml 
PRE-CREATION 
  lens-cube/src/test/resources/schema/dimensions/citydim.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimensions/countrydim.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimensions/cycledim1.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimensions/cycledim2.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimensions/daydim.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimensions/hourdim.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimensions/sports.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimensions/statedim.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimensions/testdim2.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimensions/testdim3.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimensions/testdim4.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimensions/unreachabledim.xml 
PRE-CREATION 
  lens-cube/src/test/resources/schema/dimensions/user_interests.xml 
PRE-CREATION 
  lens-cube/src/test/resources/schema/dimensions/userdim.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimensions/zipdim.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimtables/citytable.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimtables/citytable2.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimtables/citytable3.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimtables/citytable4.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimtables/countrytable.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimtables/countrytable_partitioned.xml 
PRE-CREATION 
  lens-cube/src/test/resources/schema/dimtables/cycledim1tbl.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimtables/cycledim2tbl.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimtables/daydimtbl.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimtables/hourdimtbl.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimtables/sports_tbl.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimtables/statetable.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimtables/statetable_partitioned.xml 
PRE-CREATION 
  lens-cube/src/test/resources/schema/dimtables/testdim2tbl.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimtables/testdim2tbl2.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimtables/testdim2tbl3.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimtables/testdim3tbl.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimtables/testdim4tbl.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimtables/unreachabledimtable.xml 
PRE-CREATION 
  lens-cube/src/test/resources/schema/dimtables/user_interests_tbl.xml 
PRE-CREATION 
  lens-cube/src/test/resources/schema/dimtables/usertable.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/dimtables/ziptable.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/facts/cheapfact.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/facts/summary1.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/facts/summary2.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/facts/summary3.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/facts/summary4.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/facts/testfact.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/facts/testfact1_base.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/facts/testfact1_raw_base.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/facts/testfact2.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/facts/testfact2_base.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/facts/testfact2_raw.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/facts/testfact2_raw_base.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/facts/testfact3_base.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/facts/testfact3_raw_base.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/facts/testfact4_raw_base.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/facts/testfact5_base.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/facts/testfact5_raw_base.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/facts/testfact6_base.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/facts/testfact_continuous.xml 
PRE-CREATION 
  lens-cube/src/test/resources/schema/facts/testfact_deprecated.xml 
PRE-CREATION 
  lens-cube/src/test/resources/schema/facts/testfactmonthly.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/facts/union_join_ctx_fact1.xml 
PRE-CREATION 
  lens-cube/src/test/resources/schema/facts/union_join_ctx_fact2.xml 
PRE-CREATION 
  lens-cube/src/test/resources/schema/facts/union_join_ctx_fact3.xml 
PRE-CREATION 
  lens-cube/src/test/resources/schema/facts/union_join_ctx_fact5.xml 
PRE-CREATION 
  lens-cube/src/test/resources/schema/facts/union_join_ctx_fact6.xml 
PRE-CREATION 
  lens-cube/src/test/resources/schema/segmentations/seg1.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/storages/c0.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/storages/c1.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/storages/c2.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/storages/c3.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/storages/c4.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/storages/c5.xml PRE-CREATION 
  lens-cube/src/test/resources/schema/storages/c99.xml PRE-CREATION 
  
lens-server/src/main/java/org/apache/lens/server/metastore/CubeMetastoreServiceImpl.java
 24660e1 
  lens-server/src/main/java/org/apache/lens/server/metastore/JAXBUtils.java 
7d54c7b 
  
lens-server/src/main/java/org/apache/lens/server/metastore/MetastoreResource.java
 9d823da 
  lens-server/src/test/java/org/apache/lens/server/common/RestAPITestUtil.java 
02e2f8b 
  lens-server/src/test/java/org/apache/lens/server/common/TestDataUtils.java 
0400519 
  
lens-server/src/test/java/org/apache/lens/server/query/QueryAPIErrorResponseTest.java
 567c929 


Diff: https://reviews.apache.org/r/58102/diff/5/

Changes: https://reviews.apache.org/r/58102/diff/4-5/


Testing
-------


Thanks,

Sushil Mohanty

Reply via email to