SOLR-13279: Clarify ScheduledTrigger's "every parameter missing" error response.

This commit is contained in:
Christine Poerschke 2019-06-28 18:32:25 +01:00
parent d54555c757
commit 993c051a0e
2 changed files with 4 additions and 2 deletions

View File

@ -215,6 +215,8 @@ Other Changes
* SOLR-12988: SSLTestConfig has been changed to throw AssumptionViolatedException when tests/seeds * 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) 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 ================== ================== 8.1.2 ==================
Consult the LUCENE_CHANGES.txt file for additional, low level, changes in this release. Consult the LUCENE_CHANGES.txt file for additional, low level, changes in this release.

View File

@ -65,8 +65,8 @@ public class ScheduledTrigger extends TriggerBase {
public ScheduledTrigger(String name) { public ScheduledTrigger(String name) {
super(TriggerEventType.SCHEDULED, name); super(TriggerEventType.SCHEDULED, name);
TriggerUtils.requiredProperties(requiredProperties, validProperties, "startTime"); TriggerUtils.requiredProperties(requiredProperties, validProperties, "startTime", "every");
TriggerUtils.validProperties(validProperties, "timeZone", "every", "graceDuration", AutoScalingParams.PREFERRED_OP); TriggerUtils.validProperties(validProperties, "timeZone", "graceDuration", AutoScalingParams.PREFERRED_OP);
} }
@Override @Override