> On Feb. 17, 2016, 3:31 p.m., Amareshwari Sriramadasu wrote: > > lens-driver-jdbc/src/main/java/org/apache/lens/driver/jdbc/ColumnarSQLRewriter.java, > > line 772 > > <https://reviews.apache.org/r/43600/diff/3/?file=1252337#file1252337line772> > > > > What is regex change for?
The existing code had a limitation which was causing test cases to fail after the change in `HQLParser#getString`. It was converting things like `sum(avg(expr))` to `sum(alias0)` etc. But it was relying only on string replacement, not regex replacement. So things like `sum ( avg (expr ) )` would not get replaced. The simple enhancement allows the code to cover more cases including the problem introduced by my code change. - Rajat ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43600/#review119432 ----------------------------------------------------------- On Feb. 17, 2016, 3:26 p.m., Rajat Khandelwal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/43600/ > ----------------------------------------------------------- > > (Updated Feb. 17, 2016, 3:26 p.m.) > > > Review request for lens. > > > Bugs: LENS-960 > https://issues.apache.org/jira/browse/LENS-960 > > > Repository: lens > > > Description > ------- > > Current scenario: > {noformat} > toString(parse("a or b or c or d")) == "(a or (b or (c or (d))))" > {noformat} > > Proposal is to remove the excessive nested braces. > > > Diffs > ----- > > lens-cube/src/main/java/org/apache/lens/cube/parse/HQLParser.java > fdef3f1336534904bddaf430240d9c8e163d66ee > lens-cube/src/test/java/org/apache/lens/cube/parse/TestBaseCubeQueries.java > 1d275589f43572a82b1fdb12a6606cc441b2ee37 > lens-cube/src/test/java/org/apache/lens/cube/parse/TestCubeRewriter.java > 45b43348c472778af6a118ec38982ee13ab33cf6 > > lens-cube/src/test/java/org/apache/lens/cube/parse/TestExpressionContext.java > 669a8e94d312bf58a2e559a935435da6f7d972c9 > lens-cube/src/test/java/org/apache/lens/cube/parse/TestHQLParser.java > 4afd4037e4c969796a0c7a25cd32438a7d4e7b8f > > lens-driver-jdbc/src/main/java/org/apache/lens/driver/jdbc/ColumnarSQLRewriter.java > 561452747da67feeab1f08fddbc8615324d2737f > > lens-driver-jdbc/src/test/java/org/apache/lens/driver/jdbc/TestColumnarSQLRewriter.java > c412cf0701c5c0ca70aae14e48143d509a143254 > > Diff: https://reviews.apache.org/r/43600/diff/ > > > Testing > ------- > > [INFO] > ------------------------------------------------------------------------ > [INFO] Reactor Summary: > [INFO] > [INFO] Lens Checkstyle Rules ............................. SUCCESS [2.913s] > [INFO] Lens .............................................. SUCCESS [2.229s] > [INFO] Lens API .......................................... SUCCESS [14.912s] > [INFO] Lens API for server and extensions ................ SUCCESS [8.950s] > [INFO] Lens Cube ......................................... SUCCESS > [13:17.897s] > [INFO] Lens DB storage ................................... SUCCESS [17.836s] > [INFO] Lens Query Library ................................ SUCCESS [8.737s] > [INFO] Lens Hive Driver .................................. SUCCESS [3:50.534s] > [INFO] Lens Driver for JDBC .............................. SUCCESS [29.858s] > [INFO] Lens Elastic Search Driver ........................ SUCCESS [7.778s] > [INFO] Lens Server ....................................... SUCCESS > [16:18.667s] > [INFO] Lens client ....................................... SUCCESS [23.100s] > [INFO] Lens CLI .......................................... SUCCESS [43.857s] > [INFO] Lens Examples ..................................... SUCCESS [3.091s] > [INFO] Lens Ship Jars to Distributed Cache ............... SUCCESS [0.376s] > [INFO] Lens Distribution ................................. SUCCESS [9.631s] > [INFO] Lens ML Lib ....................................... SUCCESS [1:08.303s] > [INFO] Lens ML Ext Distribution .......................... SUCCESS [1.929s] > [INFO] Lens Regression ................................... SUCCESS [2.680s] > [INFO] Lens UI ........................................... SUCCESS [30.230s] > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 38:04.886s > [INFO] Finished at: Wed Feb 17 09:43:49 UTC 2016 > [INFO] Final Memory: 207M/1950M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Rajat Khandelwal > >
