----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38402/#review99170 -----------------------------------------------------------
lens-cube/src/main/java/org/apache/lens/cube/parse/JoinResolver.java (line 582) <https://reviews.apache.org/r/38402/#comment156075> We should make this function configurable and can pass different function names from drivers. - Amareshwari Sriramadasu On Sept. 15, 2015, 12:48 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, 12:48 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 > >
