----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38402/#review99024 -----------------------------------------------------------
lens-cube/src/main/java/org/apache/lens/cube/parse/JoinResolver.java (line 556) <https://reviews.apache.org/r/38402/#comment155836> Let's check blankness of `bridgeFilterClause` instead of checking blankness of the only thing appended to `bridgeFilterClause`. lens-cube/src/main/java/org/apache/lens/cube/parse/JoinResolver.java (line 572) <https://reviews.apache.org/r/38402/#comment155837> blankness of `bridgeFromClause` lens-cube/src/main/java/org/apache/lens/cube/parse/JoinResolver.java (line 578) <https://reviews.apache.org/r/38402/#comment155838> Among the approaches: 1. Relying on a flag to append "select" or "join" 2. Making a list and later doing "select" + StringUtils.join("join", list) the second seems more intuitive. And would be easier to debug later on. lens-cube/src/main/java/org/apache/lens/cube/parse/JoinResolver.java (lines 579 - 580) <https://reviews.apache.org/r/38402/#comment155839> only null check needed. - Rajat Khandelwal On Sept. 15, 2015, 6:18 p.m., Amareshwari Sriramadasu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38402/ > ----------------------------------------------------------- > > (Updated Sept. 15, 2015, 6:18 p.m.) > > > Review request for lens. > > > Repository: lens > > > Description > ------- > > Changes include : > - Ability to specify if the destination link in a join chain has > many-many-relationship > - Ability to configure at query to say flatten the fields coming from > bridge-table relations > - Enhance cube query rewriter (mainly the join clause) to add join for bridge > table to flatten them > > Pending : > - Add a test with multi fact query > - Update documentation with feature addition > > > Diffs > ----- > > lens-api/src/main/resources/cube-0.1.xsd 58f68f5 > lens-cube/src/main/java/org/apache/lens/cube/metadata/JoinChain.java > e394e20 > lens-cube/src/main/java/org/apache/lens/cube/metadata/MetastoreUtil.java > bf27b99 > lens-cube/src/main/java/org/apache/lens/cube/metadata/SchemaGraph.java > 1a37e80 > lens-cube/src/main/java/org/apache/lens/cube/metadata/TableReference.java > 31fd97b > lens-cube/src/main/java/org/apache/lens/cube/parse/CubeQueryConfUtil.java > aab2488 > lens-cube/src/main/java/org/apache/lens/cube/parse/JoinResolver.java > c7f1e2a > lens-cube/src/test/java/org/apache/lens/cube/parse/CubeTestSetup.java > 7f56292 > lens-cube/src/test/java/org/apache/lens/cube/parse/TestJoinResolver.java > cb63fad > lens-server/src/main/java/org/apache/lens/server/metastore/JAXBUtils.java > ed472f6 > > lens-server/src/test/java/org/apache/lens/server/metastore/TestMetastoreService.java > f9be6e6 > > Diff: https://reviews.apache.org/r/38402/diff/ > > > Testing > ------- > > Added testcases for bridge tables and queries on them. > > All tests in lens-cube module pass. Will update full test results once done. > > > Thanks, > > Amareshwari Sriramadasu > >