You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shardingsphere.apache.org by me...@apache.org on 2020/07/21 01:44:09 UTC
[shardingsphere-elasticjob-ui] 13/21: wrap the boolean response
value
This is an automated email from the ASF dual-hosted git repository.
menghaoran pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/shardingsphere-elasticjob-ui.git
commit 3ef8db88aa75ba1042b0d5fcfc89c5e94b9af78c
Author: renfufei <re...@huobi.com>
AuthorDate: Mon Jul 20 20:40:44 2020 +0800
wrap the boolean response value
---
.../ui/web/controller/JobConfigController.java | 6 ++++--
.../ui/web/controller/JobOperationController.java | 18 ++++++++++++------
2 files changed, 16 insertions(+), 8 deletions(-)
diff --git a/shardingsphere-elasticjob-ui-backend/src/main/java/org/apache/shardingsphere/elasticjob/ui/web/controller/JobConfigController.java b/shardingsphere-elasticjob-ui-backend/src/main/java/org/apache/shardingsphere/elasticjob/ui/web/controller/JobConfigController.java
index 37c0ca8..ad8b76e 100644
--- a/shardingsphere-elasticjob-ui-backend/src/main/java/org/apache/shardingsphere/elasticjob/ui/web/controller/JobConfigController.java
+++ b/shardingsphere-elasticjob-ui-backend/src/main/java/org/apache/shardingsphere/elasticjob/ui/web/controller/JobConfigController.java
@@ -64,8 +64,9 @@ public final class JobConfigController {
* @param jobConfiguration job configuration
*/
@PutMapping(consumes = MediaType.APPLICATION_JSON)
- public void updateJobConfig(@RequestBody final JobConfigurationPOJO jobConfiguration) {
+ public ResponseResult<Boolean> updateJobConfig(@RequestBody final JobConfigurationPOJO jobConfiguration) {
jobAPIService.getJobConfigurationAPI().updateJobConfiguration(jobConfiguration);
+ return ResponseResultUtil.build(Boolean.TRUE);
}
/**
@@ -74,7 +75,8 @@ public final class JobConfigController {
* @param jobName job name
*/
@DeleteMapping("/{jobName}")
- public void removeJob(@PathVariable("jobName") final String jobName) {
+ public ResponseResult<Boolean> removeJob(@PathVariable("jobName") final String jobName) {
jobAPIService.getJobConfigurationAPI().removeJobConfiguration(jobName);
+ return ResponseResultUtil.build(Boolean.TRUE);
}
}
diff --git a/shardingsphere-elasticjob-ui-backend/src/main/java/org/apache/shardingsphere/elasticjob/ui/web/controller/JobOperationController.java b/shardingsphere-elasticjob-ui-backend/src/main/java/org/apache/shardingsphere/elasticjob/ui/web/controller/JobOperationController.java
index 32fde53..6c79d9a 100644
--- a/shardingsphere-elasticjob-ui-backend/src/main/java/org/apache/shardingsphere/elasticjob/ui/web/controller/JobOperationController.java
+++ b/shardingsphere-elasticjob-ui-backend/src/main/java/org/apache/shardingsphere/elasticjob/ui/web/controller/JobOperationController.java
@@ -73,8 +73,9 @@ public final class JobOperationController {
* @param jobName job name
*/
@PostMapping("/{jobName}/trigger")
- public void triggerJob(@PathVariable("jobName") final String jobName) {
+ public ResponseResult<Boolean> triggerJob(@PathVariable("jobName") final String jobName) {
jobAPIService.getJobOperatorAPI().trigger(jobName);
+ return ResponseResultUtil.build(Boolean.TRUE);
}
/**
@@ -83,8 +84,9 @@ public final class JobOperationController {
* @param jobName job name
*/
@PostMapping(value = "/{jobName}/disable")
- public void disableJob(@PathVariable("jobName") final String jobName) {
+ public ResponseResult<Boolean> disableJob(@PathVariable("jobName") final String jobName) {
jobAPIService.getJobOperatorAPI().disable(jobName, null);
+ return ResponseResultUtil.build(Boolean.TRUE);
}
/**
@@ -93,8 +95,9 @@ public final class JobOperationController {
* @param jobName job name
*/
@PostMapping(value = "/{jobName}/enable")
- public void enableJob(@PathVariable("jobName") final String jobName) {
+ public ResponseResult<Boolean> enableJob(@PathVariable("jobName") final String jobName) {
jobAPIService.getJobOperatorAPI().enable(jobName, null);
+ return ResponseResultUtil.build(Boolean.TRUE);
}
/**
@@ -103,8 +106,9 @@ public final class JobOperationController {
* @param jobName job name
*/
@PostMapping(value = "/{jobName}/shutdown")
- public void shutdownJob(@PathVariable("jobName") final String jobName) {
+ public ResponseResult<Boolean> shutdownJob(@PathVariable("jobName") final String jobName) {
jobAPIService.getJobOperatorAPI().shutdown(jobName, null);
+ return ResponseResultUtil.build(Boolean.TRUE);
}
/**
@@ -126,8 +130,9 @@ public final class JobOperationController {
* @param item sharding item
*/
@PostMapping(value = "/{jobName}/sharding/{item}/disable")
- public void disableSharding(@PathVariable("jobName") final String jobName, @PathVariable("item") final String item) {
+ public ResponseResult<Boolean> disableSharding(@PathVariable("jobName") final String jobName, @PathVariable("item") final String item) {
jobAPIService.getShardingOperateAPI().disable(jobName, item);
+ return ResponseResultUtil.build(Boolean.TRUE);
}
/**
@@ -137,7 +142,8 @@ public final class JobOperationController {
* @param item sharding item
*/
@PostMapping(value = "/{jobName}/sharding/{item}/enable")
- public void enableSharding(@PathVariable("jobName") final String jobName, @PathVariable("item") final String item) {
+ public ResponseResult<Boolean> enableSharding(@PathVariable("jobName") final String jobName, @PathVariable("item") final String item) {
jobAPIService.getShardingOperateAPI().enable(jobName, item);
+ return ResponseResultUtil.build(Boolean.TRUE);
}
}