> On April 20, 2016, 10:14 a.m., Rajat Khandelwal wrote: > > lens-cube/src/test/java/org/apache/lens/cube/parse/TestBaseCubeQueries.java, > > lines 485-486 > > <https://reviews.apache.org/r/46430/diff/1/?file=1353062#file1353062line485> > > > > Should the outer alias be `func1(dim1)` and `func2(dim1)` instead of > > `expr1` and `expr2`?
Didnt want to make any such - as we are not sure all drivers might support such aliases. Let user give is he want any alias. If not given, we are giving alias with index of the selection. - Amareshwari ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46430/#review129718 ----------------------------------------------------------- On April 20, 2016, 10:05 a.m., Amareshwari Sriramadasu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46430/ > ----------------------------------------------------------- > > (Updated April 20, 2016, 10:05 a.m.) > > > Review request for lens. > > > Bugs: LENS-730 > https://issues.apache.org/jira/browse/LENS-730 > > > Repository: lens > > > Description > ------- > > Fixes following alias issues with multi fact queries and expressions > > - When alias of a selected expression is a column name used in a Expression > column. For ex : select col1 as f1, expr1 as f2 from cube ; expr1 = > complex(f2) > - When different expressions (functions) are around a column and no alias is > passed for them : For : func1(dim1), func2(dim1) > - When alias is in the form of expression. For ex: select msr1 as `sum(msr1)` > from cube > > > Diffs > ----- > > lens-cube/src/main/java/org/apache/lens/cube/parse/AggregateResolver.java > fd7036a > lens-cube/src/main/java/org/apache/lens/cube/parse/ColumnResolver.java > 75aa3f4 > lens-cube/src/main/java/org/apache/lens/cube/parse/ExpressionResolver.java > 04c3c45 > lens-cube/src/test/java/org/apache/lens/cube/parse/CubeTestSetup.java > 379e873 > lens-cube/src/test/java/org/apache/lens/cube/parse/TestBaseCubeQueries.java > fec5d51 > > lens-cube/src/test/java/org/apache/lens/cube/parse/TestBridgeTableQueries.java > e8905c1 > lens-cube/src/test/java/org/apache/lens/cube/parse/TestCubeRewriter.java > 6ee45d8 > > lens-cube/src/test/java/org/apache/lens/cube/parse/TestExpressionResolver.java > fa07b67 > > Diff: https://reviews.apache.org/r/46430/diff/ > > > Testing > ------- > > All cube tests pass. > > > Thanks, > > Amareshwari Sriramadasu > >
