> On Dec. 29, 2014, 3:53 p.m., Amareshwari Sriramadasu wrote: > > lens-cube/src/main/java/org/apache/lens/cube/metadata/ExprColumn.java, line > > 47 > > <https://reviews.apache.org/r/29451/diff/2/?file=802259#file802259line47> > > > > We cannot do lowercase. There might be literal in expression.
Updatd patch. - Jaideep ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29451/#review66247 ----------------------------------------------------------- On Dec. 30, 2014, 5:40 a.m., Jaideep dhok wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29451/ > ----------------------------------------------------------- > > (Updated Dec. 30, 2014, 5:40 a.m.) > > > Review request for lens. > > > Bugs: LENS-175 > https://issues.apache.org/jira/browse/LENS-175 > > > Repository: lens > > > Description > ------- > > Changes in ExprColumn and test changes to validate multiple expressions. No > changes required in CubeMetastoreClient > > > Diffs > ----- > > lens-cube/src/main/java/org/apache/lens/cube/metadata/ExprColumn.java > 352fef9 > lens-cube/src/main/java/org/apache/lens/cube/parse/HQLParser.java 7347b3a > > lens-cube/src/test/java/org/apache/lens/cube/metadata/TestCubeMetastoreClient.java > 5889506 > lens-cube/src/test/java/org/apache/lens/cube/metadata/TestExprColumn.java > PRE-CREATION > lens-cube/src/test/java/org/apache/lens/cube/parse/TestHQLParser.java > 760aaab > > Diff: https://reviews.apache.org/r/29451/diff/ > > > Testing > ------- > > Added new tests for creating and altering multiple expressions. > No impact on existing lens-cube tests. I'll update JIRA with complete test > report. > > > Thanks, > > Jaideep dhok > >
