You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by cp...@apache.org on 2019/06/28 17:59:08 UTC

[lucene-solr] 01/03: SOLR-13279: Clarify ScheduledTrigger's "every parameter missing" error response.

This is an automated email from the ASF dual-hosted git repository.

cpoerschke pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/lucene-solr.git

commit 993c051a0edb6d634396fd814f0e2bb80d65ce29
Author: Christine Poerschke <cp...@apache.org>
AuthorDate: Fri Jun 28 18:32:25 2019 +0100

    SOLR-13279: Clarify ScheduledTrigger's "every parameter missing" error response.
---
 solr/CHANGES.txt                                                      | 2 ++
 .../src/java/org/apache/solr/cloud/autoscaling/ScheduledTrigger.java  | 4 ++--
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/solr/CHANGES.txt b/solr/CHANGES.txt
index 5485927..ab99176 100644
--- a/solr/CHANGES.txt
+++ b/solr/CHANGES.txt
@@ -215,6 +215,8 @@ Other Changes
 * SOLR-12988: SSLTestConfig has been changed to throw AssumptionViolatedException when tests/seeds
   request SSL but the JVM appears to be an OpenJDK version known to have SSL bugs (hossman, Cao Manh Dat)
 
+* SOLR-13279: Clarify ScheduledTrigger's "every parameter missing" error response. (Christine Poerschke)
+
 ==================  8.1.2 ==================
 
 Consult the LUCENE_CHANGES.txt file for additional, low level, changes in this release.
diff --git a/solr/core/src/java/org/apache/solr/cloud/autoscaling/ScheduledTrigger.java b/solr/core/src/java/org/apache/solr/cloud/autoscaling/ScheduledTrigger.java
index e5afd9f..c1fe040 100644
--- a/solr/core/src/java/org/apache/solr/cloud/autoscaling/ScheduledTrigger.java
+++ b/solr/core/src/java/org/apache/solr/cloud/autoscaling/ScheduledTrigger.java
@@ -65,8 +65,8 @@ public class ScheduledTrigger extends TriggerBase {
 
   public ScheduledTrigger(String name) {
     super(TriggerEventType.SCHEDULED, name);
-    TriggerUtils.requiredProperties(requiredProperties, validProperties, "startTime");
-    TriggerUtils.validProperties(validProperties, "timeZone", "every", "graceDuration", AutoScalingParams.PREFERRED_OP);
+    TriggerUtils.requiredProperties(requiredProperties, validProperties, "startTime", "every");
+    TriggerUtils.validProperties(validProperties, "timeZone", "graceDuration", AutoScalingParams.PREFERRED_OP);
   }
 
   @Override