----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29422/ -----------------------------------------------------------
(Updated Dec. 26, 2014, 12:49 p.m.) Review request for lens. Changes ------- (1) Changed logic to retrieve select expressions without alias form AST instead of dropping alias suffix. (2) Changed test logging level to WARN (3) Added a unit test without "as" keyword between column and alias. Repository: lens Description ------- (1) getExpressionWithoutAlias in GroupbyResolver.class is unable to handle aliases with spaces in them. Fixed the same. (2) Also improved log4j.properties of lens-cube to send logs to both console appender and test log files. (3) Used Slf4j annotation in Test classes for getting a reference to Slf4j logger. Added required dependencies in lens-cube/pom.xml to facilitate the same. Diffs (updated) ----- lens-cube/pom.xml 1bcb4dc lens-cube/src/main/java/org/apache/lens/cube/parse/GroupbyResolver.java c2fef7e lens-cube/src/main/java/org/apache/lens/cube/parse/HQLParser.java 8e41830 lens-cube/src/test/java/org/apache/lens/cube/parse/CubeTestSetup.java f515df1 lens-cube/src/test/java/org/apache/lens/cube/parse/TestCubeRewriter.java ad4abcf lens-cube/src/test/java/org/apache/lens/cube/parse/TestQueryRewrite.java fd939e9 lens-cube/src/test/resources/log4j.properties 9729de0 Diff: https://reviews.apache.org/r/29422/diff/ Testing (updated) ------- Added new unit test cases for the changes. All existing Unit Test Cases passed. Thanks, Himanshu Gahlaut
