update exception handling and restful response 1. Update exception handling. Add exception handlers for client error and checked exception at server side. Remove unnecessary try-catch for runtime exception. 2. Modify HTTP response according to restful principle, for exception response, add HTTP status code and customized error code and message into the response body. 3. Change the implement of 'add metric values' method in MetricStore to use bulk api of elasticsearch
Author: He Wang <[email protected]> Closes #201 from whhe/master. Project: http://git-wip-us.apache.org/repos/asf/incubator-griffin/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-griffin/commit/6b1a7000 Tree: http://git-wip-us.apache.org/repos/asf/incubator-griffin/tree/6b1a7000 Diff: http://git-wip-us.apache.org/repos/asf/incubator-griffin/diff/6b1a7000 Branch: refs/heads/master Commit: 6b1a7000d1ccfd9225b3a02e23df90060f2a1aec Parents: 2972aa2 Author: He Wang <[email protected]> Authored: Tue Jan 30 09:45:06 2018 +0800 Committer: Lionel Liu <[email protected]> Committed: Tue Jan 30 09:45:06 2018 +0800 ---------------------------------------------------------------------- griffin-doc/service/postman/griffin.json | 2457 +++++++++--------- .../core/error/exception/GriffinException.java | 41 - .../exception/RuntimeExceptionHandler.java | 62 - .../core/exception/GriffinException.java | 62 + .../core/exception/GriffinExceptionHandler.java | 52 + .../core/exception/GriffinExceptionMessage.java | 77 + .../exception/GriffinExceptionResponse.java | 82 + .../apache/griffin/core/job/JobController.java | 20 +- .../org/apache/griffin/core/job/JobService.java | 14 +- .../apache/griffin/core/job/JobServiceImpl.java | 145 +- .../apache/griffin/core/job/SparkSubmitJob.java | 7 +- .../measure/ExternalMeasureOperationImpl.java | 65 +- .../measure/GriffinMeasureOperationImpl.java | 52 +- .../griffin/core/measure/MeasureController.java | 15 +- .../griffin/core/measure/MeasureOperation.java | 7 +- .../griffin/core/measure/MeasureService.java | 7 +- .../core/measure/MeasureServiceImpl.java | 30 +- .../hive/HiveMetaStoreServiceImpl.java | 10 +- .../griffin/core/metric/MetricController.java | 7 +- .../griffin/core/metric/MetricService.java | 5 +- .../griffin/core/metric/MetricServiceImpl.java | 69 +- .../apache/griffin/core/metric/MetricStore.java | 8 +- .../griffin/core/metric/MetricStoreImpl.java | 71 +- .../core/util/GriffinOperationMessage.java | 79 - .../griffin/core/job/JobControllerTest.java | 122 +- .../griffin/core/job/JobServiceImplTest.java | 312 ++- .../griffin/core/job/SparkSubmitJobTest.java | 22 +- .../ExternalMeasureOperationImplTest.java | 102 + .../GriffinMeasureOperationImplTest.java | 120 + .../core/measure/MeasureControllerTest.java | 93 +- .../core/measure/MeasureOrgControllerTest.java | 6 - .../core/measure/MeasureServiceImplTest.java | 259 +- .../core/metric/MetricControllerTest.java | 172 ++ .../core/metric/MetricServiceImplTest.java | 172 ++ .../apache/griffin/core/util/EntityHelper.java | 5 + .../apache/griffin/core/util/TimeUtilTest.java | 35 +- 36 files changed, 2741 insertions(+), 2123 deletions(-) ----------------------------------------------------------------------
