> On July 28, 2015, 6:19 a.m., Rajat Khandelwal wrote: > > lens-cube/src/main/java/org/apache/lens/cube/parse/ContextRewriter.java, > > line 34 > > <https://reviews.apache.org/r/36850/diff/1/?file=1022465#file1022465line34> > > > > I believe all the semantic exceptions should have been gone. If not, > > can you elaborate on why? > > Rajat Khandelwal wrote: > Also, let's do a `git grep` and make sure they are not used anymore.
Classes which extends Hive SemanticAnalyzer will continue throwing SematicException. > On July 28, 2015, 6:19 a.m., Rajat Khandelwal wrote: > > lens-cube/src/main/java/org/apache/lens/cube/parse/CubeQueryRewriter.java, > > line 197 > > <https://reviews.apache.org/r/36850/diff/1/?file=1022467#file1022467line197> > > > > why hive exception? Fixed to throw only LensExceotion. > On July 28, 2015, 6:19 a.m., Rajat Khandelwal wrote: > > lens-cube/src/main/java/org/apache/lens/cube/parse/JoinResolver.java, line > > 980 > > <https://reviews.apache.org/r/36850/diff/1/?file=1022478#file1022478line980> > > > > is it throwing `HiveException` anymore? No, chnaged accordingly. - Sushil ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36850/#review93241 ----------------------------------------------------------- On Aug. 17, 2015, 9:19 a.m., Sushil Mohanty wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36850/ > ----------------------------------------------------------- > > (Updated Aug. 17, 2015, 9:19 a.m.) > > > Review request for lens, Amareshwari Sriramadasu and Himanshu Gahlaut. > > > Bugs: LENS-187 > https://issues.apache.org/jira/browse/LENS-187 > > > Repository: lens > > > Description > ------- > > Moved all cube related error messages from (InMobi) Hive to Lens. > > > Diffs > ----- > > lens-api/src/main/resources/lens-errors.conf e6715b6 > lens-cube/src/main/java/org/apache/lens/cube/error/LensCubeErrorCode.java > 0006b22 > > lens-cube/src/main/java/org/apache/lens/cube/parse/AbridgedTimeRangeWriter.java > d5276e0 > lens-cube/src/main/java/org/apache/lens/cube/parse/AggregateResolver.java > f880495 > lens-cube/src/main/java/org/apache/lens/cube/parse/AliasReplacer.java > 9309307 > > lens-cube/src/main/java/org/apache/lens/cube/parse/BetweenTimeRangeWriter.java > 6c85c2d > lens-cube/src/main/java/org/apache/lens/cube/parse/CandidateFact.java > 6c88fb3 > > lens-cube/src/main/java/org/apache/lens/cube/parse/CandidateTableResolver.java > 3e73d02 > lens-cube/src/main/java/org/apache/lens/cube/parse/CheckColumnMapping.java > 59e4fd3 > lens-cube/src/main/java/org/apache/lens/cube/parse/CheckTableNames.java > fce1a04 > lens-cube/src/main/java/org/apache/lens/cube/parse/ColumnResolver.java > 2ff5959 > lens-cube/src/main/java/org/apache/lens/cube/parse/ContextRewriter.java > feb26d7 > lens-cube/src/main/java/org/apache/lens/cube/parse/CubeQueryContext.java > d753e3f > lens-cube/src/main/java/org/apache/lens/cube/parse/CubeQueryRewriter.java > 72dc64b > > lens-cube/src/main/java/org/apache/lens/cube/parse/CubeSemanticAnalyzer.java > 68bffed > lens-cube/src/main/java/org/apache/lens/cube/parse/DateUtil.java 5c77548 > > lens-cube/src/main/java/org/apache/lens/cube/parse/DenormalizationResolver.java > 517e8fc > lens-cube/src/main/java/org/apache/lens/cube/parse/DimHQLContext.java > 3814cf6 > lens-cube/src/main/java/org/apache/lens/cube/parse/DimOnlyHQLContext.java > 922501d > lens-cube/src/main/java/org/apache/lens/cube/parse/ExpressionResolver.java > 5bdb412 > lens-cube/src/main/java/org/apache/lens/cube/parse/FactHQLContext.java > 623c58b > lens-cube/src/main/java/org/apache/lens/cube/parse/FieldValidator.java > 1a1232b > lens-cube/src/main/java/org/apache/lens/cube/parse/GroupbyResolver.java > 4d2692b > lens-cube/src/main/java/org/apache/lens/cube/parse/HQLContextInterface.java > 35011e8 > lens-cube/src/main/java/org/apache/lens/cube/parse/HQLParser.java 7b99310 > lens-cube/src/main/java/org/apache/lens/cube/parse/JoinResolver.java > a6e9340 > > lens-cube/src/main/java/org/apache/lens/cube/parse/LeastPartitionResolver.java > 7f02ae8 > > lens-cube/src/main/java/org/apache/lens/cube/parse/LightestDimensionResolver.java > 4ae6226 > > lens-cube/src/main/java/org/apache/lens/cube/parse/LightestFactResolver.java > fba682d > > lens-cube/src/main/java/org/apache/lens/cube/parse/MaxCoveringFactResolver.java > 4d8cbf3 > lens-cube/src/main/java/org/apache/lens/cube/parse/MultiFactHQLContext.java > b5f5adc > lens-cube/src/main/java/org/apache/lens/cube/parse/SimpleHQLContext.java > f3f3f78 > > lens-cube/src/main/java/org/apache/lens/cube/parse/SingleFactHQLContext.java > b63111b > > lens-cube/src/main/java/org/apache/lens/cube/parse/SingleFactMultiStorageHQLContext.java > 4ad2f1f > > lens-cube/src/main/java/org/apache/lens/cube/parse/StorageTableResolver.java > aa76c0c > lens-cube/src/main/java/org/apache/lens/cube/parse/TimeRange.java 03732cb > lens-cube/src/main/java/org/apache/lens/cube/parse/TimeRangeWriter.java > 12acf98 > lens-cube/src/main/java/org/apache/lens/cube/parse/TimerangeResolver.java > 91c0c75 > lens-cube/src/main/java/org/apache/lens/cube/parse/UnionHQLContext.java > e6ed86b > lens-cube/src/main/java/org/apache/lens/cube/parse/ValidationRule.java > 9567845 > lens-cube/src/test/java/org/apache/lens/cube/parse/CubeTestSetup.java > ae8984f > > lens-cube/src/test/java/org/apache/lens/cube/parse/FieldsCannotBeQueriedTogetherTest.java > 501a4b0 > > lens-cube/src/test/java/org/apache/lens/cube/parse/TestAggregateResolver.java > 13058e2 > lens-cube/src/test/java/org/apache/lens/cube/parse/TestBaseCubeQueries.java > 0d0b927 > lens-cube/src/test/java/org/apache/lens/cube/parse/TestCubeRewriter.java > fb8f61a > lens-cube/src/test/java/org/apache/lens/cube/parse/TestDateUtil.java > 6717b5b > > lens-cube/src/test/java/org/apache/lens/cube/parse/TestDenormalizationResolver.java > 1bf1a5c > > lens-cube/src/test/java/org/apache/lens/cube/parse/TestExpressionResolver.java > 7f872e9 > lens-cube/src/test/java/org/apache/lens/cube/parse/TestJoinResolver.java > ed08605 > lens-cube/src/test/java/org/apache/lens/cube/parse/TestQueryRewrite.java > d16f6a5 > lens-cube/src/test/java/org/apache/lens/cube/parse/TestRewriterPlan.java > 0805fb5 > > lens-cube/src/test/java/org/apache/lens/cube/parse/TestTimeRangeExtractor.java > 26836fe > > lens-cube/src/test/java/org/apache/lens/cube/parse/TestTimeRangeResolver.java > a30a114 > lens-cube/src/test/java/org/apache/lens/cube/parse/TestTimeRangeWriter.java > 453a102 > > lens-cube/src/test/java/org/apache/lens/cube/parse/TestTimeRangeWriterWithQuery.java > 8da740b > > lens-driver-jdbc/src/test/java/org/apache/lens/driver/jdbc/TestColumnarSQLRewriter.java > 9222d87 > > lens-server-api/src/main/java/org/apache/lens/server/api/error/LensException.java > 123b6ee > > lens-server/src/main/java/org/apache/lens/server/error/UnSupportedQuerySubmitOpException.java > 4ab26ec > lens-server/src/main/java/org/apache/lens/server/rewrite/RewriteUtil.java > 5844010 > > lens-server/src/test/java/org/apache/lens/server/query/TestQueryService.java > 131f008 > lens-server/src/test/java/org/apache/lens/server/rewrite/TestRewriting.java > affefe1 > > Diff: https://reviews.apache.org/r/36850/diff/ > > > Testing > ------- > > All unit tests are running fine. > [INFO] > ------------------------------------------------------------------------ > [INFO] Reactor Summary: > [INFO] > [INFO] Lens Checkstyle Rules ............................. SUCCESS [18.289s] > [INFO] Lens .............................................. SUCCESS [0.371s] > [INFO] Lens API .......................................... SUCCESS [26.254s] > [INFO] Lens API for server and extensions ................ SUCCESS [26.354s] > [INFO] Lens Cube ......................................... SUCCESS [5:24.863s] > [INFO] Lens DB storage ................................... SUCCESS [20.037s] > [INFO] Lens Query Library ................................ SUCCESS [10.167s] > [INFO] Lens Hive Driver .................................. SUCCESS [3:34.907s] > [INFO] Lens Driver for JDBC .............................. SUCCESS [34.493s] > [INFO] Lens Server ....................................... SUCCESS > [19:37.403s] > [INFO] Lens client ....................................... SUCCESS [55.363s] > [INFO] Lens CLI .......................................... SUCCESS [4:31.293s] > [INFO] Lens Examples ..................................... SUCCESS [7.305s] > [INFO] Lens Distribution ................................. SUCCESS [2.314s] > [INFO] Lens ML Lib ....................................... SUCCESS > [13:28.783s] > [INFO] Lens ML Ext Distribution .......................... SUCCESS [13.554s] > [INFO] Lens Regression ................................... SUCCESS [3.790s] > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 50:17.722s > [INFO] Finished at: Mon Jul 27 21:05:04 IST 2015 > [INFO] Final Memory: 173M/1864M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Sushil Mohanty > >
