----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37648/ -----------------------------------------------------------
(Updated Aug. 27, 2015, 1:51 p.m.) Review request for lens. Changes ------- Updated the patch with weight defined for error code. Error code with heigher weight is given to user. The earlier approach of defining errocodes in sequential order is not optimal way, provided if we wish to add a new errrorcode inbetween the predefined ones. All unit tests are running fine. [INFO] ------------------------------------------------------------------------ [INFO] Reactor Summary: [INFO] [INFO] Lens Checkstyle Rules ............................. SUCCESS [2.202s] [INFO] Lens .............................................. SUCCESS [3.054s] [INFO] Lens API .......................................... SUCCESS [24.408s] [INFO] Lens API for server and extensions ................ SUCCESS [19.810s] [INFO] Lens Cube ......................................... SUCCESS [5:10.525s] [INFO] Lens DB storage ................................... SUCCESS [20.688s] [INFO] Lens Query Library ................................ SUCCESS [14.823s] [INFO] Lens Hive Driver .................................. SUCCESS [2:50.741s] [INFO] Lens Driver for JDBC .............................. SUCCESS [35.898s] [INFO] Lens Elastic Search Driver ........................ SUCCESS [18.463s] [INFO] Lens Server ....................................... SUCCESS [5:35.523s] [INFO] Lens client ....................................... SUCCESS [38.685s] [INFO] Lens CLI .......................................... SUCCESS [2:35.191s] [INFO] Lens Examples ..................................... SUCCESS [8.824s] [INFO] Lens Distribution ................................. SUCCESS [8.400s] [INFO] Lens ML Lib ....................................... SUCCESS [1:19.226s] [INFO] Lens ML Ext Distribution .......................... SUCCESS [2.071s] [INFO] Lens Regression ................................... SUCCESS [12.273s] [INFO] ------------------------------------------------------------------------ [INFO] BUILD SUCCESS [INFO] ------------------------------------------------------------------------ [INFO] Total time: 21:01.699s [INFO] Finished at: Thu Aug 27 13:27:53 UTC 2015 [INFO] Final Memory: 171M/801M [INFO] ------------------------------------------------------------------------ Repository: lens Description ------- When two drivers give two different errors, it results in Internal server error with LensMultiCauseException.It should compare both errors and give more approriate one to end user. The error codes are organized in sequential order. Error code with higher value is being thrown. Diffs (updated) ----- lens-api/src/main/resources/lens-errors.conf 3fb191e lens-cube/src/main/java/org/apache/lens/cube/error/ColUnAvailableInTimeRangeException.java dd3bb72 lens-cube/src/main/java/org/apache/lens/cube/error/FieldsCannotBeQueriedTogetherException.java 65b96d7 lens-cube/src/main/java/org/apache/lens/cube/error/LensCubeErrorCode.java 1fe74e2 lens-cube/src/main/java/org/apache/lens/cube/parse/AggregateResolver.java 9c0f936 lens-cube/src/main/java/org/apache/lens/cube/parse/AliasReplacer.java 98e38d5 lens-cube/src/main/java/org/apache/lens/cube/parse/BetweenTimeRangeWriter.java 4bd7cc8 lens-cube/src/main/java/org/apache/lens/cube/parse/CandidateTableResolver.java 69fbcc5 lens-cube/src/main/java/org/apache/lens/cube/parse/ColumnResolver.java dfe0c32 lens-cube/src/main/java/org/apache/lens/cube/parse/CubeQueryContext.java 16429f0 lens-cube/src/main/java/org/apache/lens/cube/parse/CubeQueryRewriter.java 0dfd7da lens-cube/src/main/java/org/apache/lens/cube/parse/DateUtil.java 486c6b3 lens-cube/src/main/java/org/apache/lens/cube/parse/DenormalizationResolver.java 3ef9652 lens-cube/src/main/java/org/apache/lens/cube/parse/HQLParser.java 586629f lens-cube/src/main/java/org/apache/lens/cube/parse/JoinResolver.java 826a59d lens-cube/src/main/java/org/apache/lens/cube/parse/MultiFactHQLContext.java d8515d8 lens-cube/src/main/java/org/apache/lens/cube/parse/TimeRange.java 5444e71 lens-cube/src/main/java/org/apache/lens/cube/parse/TimerangeResolver.java fb1c89e lens-cube/src/test/java/org/apache/lens/cube/parse/TestBaseCubeQueries.java 9120a70 lens-cube/src/test/java/org/apache/lens/cube/parse/TestCubeRewriter.java b34c217 lens-cube/src/test/java/org/apache/lens/cube/parse/TestExpressionResolver.java ca07a0d lens-cube/src/test/java/org/apache/lens/cube/parse/TestJoinResolver.java 6150b14 lens-cube/src/test/java/org/apache/lens/cube/parse/TestTimeRangeExtractor.java 3c3aa9c lens-cube/src/test/java/org/apache/lens/cube/parse/TestTimeRangeWriter.java e5540d2 lens-cube/src/test/java/org/apache/lens/cube/parse/TestTimeRangeWriterWithQuery.java 2083ef9 lens-driver-es/src/main/java/org/apache/lens/driver/es/exceptions/ESClientException.java c2254f1 lens-driver-es/src/main/java/org/apache/lens/driver/es/exceptions/InvalidQueryException.java 8127cba lens-server-api/src/main/java/org/apache/lens/server/api/LensErrorInfo.java PRE-CREATION lens-server-api/src/main/java/org/apache/lens/server/api/error/LensException.java 603d7cb lens-server-api/src/main/java/org/apache/lens/server/api/error/LensMultiCauseException.java af9e33a lens-server/src/main/java/org/apache/lens/server/error/LensServerErrorCode.java 917eba3 lens-server/src/main/java/org/apache/lens/server/error/UnSupportedQuerySubmitOpException.java 366b306 lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java c29a1ac lens-server/src/main/java/org/apache/lens/server/query/QueryServiceResource.java 6bfd77a lens-server/src/test/java/org/apache/lens/server/common/ErrorResponseExpectedData.java 38b5ea0 lens-server/src/test/java/org/apache/lens/server/common/FailingQueryDriver.java 64a75ce lens-server/src/test/java/org/apache/lens/server/query/QueryAPIErrorResponseTest.java 2189eb8 lens-server/src/test/java/org/apache/lens/server/query/TestQueryService.java b3f5d93 Diff: https://reviews.apache.org/r/37648/diff/ Testing ------- mvn clean install runs fine. [INFO] ------------------------------------------------------------------------ [INFO] Reactor Summary: [INFO] [INFO] Lens Checkstyle Rules ............................. SUCCESS [2.212s] [INFO] Lens .............................................. SUCCESS [3.014s] [INFO] Lens API .......................................... SUCCESS [23.728s] [INFO] Lens API for server and extensions ................ SUCCESS [18.542s] [INFO] Lens Cube ......................................... SUCCESS [5:19.711s] [INFO] Lens DB storage ................................... SUCCESS [20.030s] [INFO] Lens Query Library ................................ SUCCESS [16.741s] [INFO] Lens Hive Driver .................................. SUCCESS [2:58.268s] [INFO] Lens Driver for JDBC .............................. SUCCESS [40.291s] [INFO] Lens Elastic Search Driver ........................ SUCCESS [23.075s] [INFO] Lens Server ....................................... SUCCESS [6:18.562s] [INFO] Lens client ....................................... SUCCESS [39.919s] [INFO] Lens CLI .......................................... SUCCESS [3:22.884s] [INFO] Lens Examples ..................................... SUCCESS [10.586s] [INFO] Lens Distribution ................................. SUCCESS [10.094s] [INFO] Lens ML Lib ....................................... SUCCESS [1:32.958s] [INFO] Lens ML Ext Distribution .......................... SUCCESS [2.238s] [INFO] Lens Regression ................................... SUCCESS [14.124s] [INFO] ------------------------------------------------------------------------ [INFO] BUILD SUCCESS [INFO] ------------------------------------------------------------------------ [INFO] Total time: 23:17.878s [INFO] Finished at: Thu Aug 20 11:57:34 UTC 2015 [INFO] Final Memory: 171M/801M [INFO] ------------------------------------------------------------------------ Thanks, Sushil Mohanty
