You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@griffin.apache.org by gu...@apache.org on 2018/01/30 01:45:17 UTC
[5/5] incubator-griffin git commit: update exception handling and
restful response
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 <wa...@qq.com>
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 <wa...@qq.com>
Authored: Tue Jan 30 09:45:06 2018 +0800
Committer: Lionel Liu <bh...@163.com>
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(-)
----------------------------------------------------------------------