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));

Reply via email to