This is an automated email from the ASF dual-hosted git repository.
zhangliang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/shardingsphere-elasticjob.git
The following commit(s) were added to refs/heads/master by this push:
new a8573f5 Remove checking cron in
JobConfigurationAPIImpl#updateJobConfiguration (#1419) (#1420)
a8573f5 is described below
commit a8573f5beb089d8061ed5154659078893f6517b8
Author: Tboy <[email protected]>
AuthorDate: Wed Aug 26 23:49:23 2020 +0800
Remove checking cron in JobConfigurationAPIImpl#updateJobConfiguration
(#1419) (#1420)
---
.../lite/lifecycle/internal/settings/JobConfigurationAPIImpl.java | 1 -
1 file changed, 1 deletion(-)
diff --git
a/elasticjob-lite/elasticjob-lite-lifecycle/src/main/java/org/apache/shardingsphere/elasticjob/lite/lifecycle/internal/settings/JobConfigurationAPIImpl.java
b/elasticjob-lite/elasticjob-lite-lifecycle/src/main/java/org/apache/shardingsphere/elasticjob/lite/lifecycle/internal/settings/JobConfigurationAPIImpl.java
index a31bd78..420068d 100644
---
a/elasticjob-lite/elasticjob-lite-lifecycle/src/main/java/org/apache/shardingsphere/elasticjob/lite/lifecycle/internal/settings/JobConfigurationAPIImpl.java
+++
b/elasticjob-lite/elasticjob-lite-lifecycle/src/main/java/org/apache/shardingsphere/elasticjob/lite/lifecycle/internal/settings/JobConfigurationAPIImpl.java
@@ -42,7 +42,6 @@ public final class JobConfigurationAPIImpl implements
JobConfigurationAPI {
@Override
public void updateJobConfiguration(final JobConfigurationPOJO jobConfig) {
Preconditions.checkArgument(!Strings.isNullOrEmpty(jobConfig.getJobName()),
"jobName can not be empty.");
-
Preconditions.checkArgument(!Strings.isNullOrEmpty(jobConfig.getCron()), "cron
can not be empty.");
Preconditions.checkArgument(jobConfig.getShardingTotalCount() > 0,
"shardingTotalCount should larger than zero.");
JobNodePath jobNodePath = new JobNodePath(jobConfig.getJobName());
regCenter.update(jobNodePath.getConfigNodePath(),
YamlEngine.marshal(jobConfig));