removing requirement for lookback to be for cron only

Author: toepkerd-zz

core/src/main/kotlin/org/opensearch/alerting/core/modelv2/PPLMonitor.kt

@@ -140,13 +140,13 @@ data class PPLMonitor(
             builder.optionalUserField(USER_FIELD, user)
         }
 
-        builder.field(MonitorV2.NAME_FIELD, name)
-        builder.field(MonitorV2.SCHEDULE_FIELD, schedule)
-        builder.field(MonitorV2.LOOK_BACK_WINDOW_FIELD, lookBackWindow?.toHumanReadableString(0))
-        builder.field(MonitorV2.ENABLED_FIELD, enabled)
-        builder.nonOptionalTimeField(MonitorV2.LAST_UPDATE_TIME_FIELD, lastUpdateTime)
-        builder.optionalTimeField(MonitorV2.ENABLED_TIME_FIELD, enabledTime)
-        builder.field(MonitorV2.TRIGGERS_FIELD, triggers.toTypedArray())
+        builder.field(NAME_FIELD, name)
+        builder.field(SCHEDULE_FIELD, schedule)
+        builder.field(LOOK_BACK_WINDOW_FIELD, lookBackWindow?.toHumanReadableString(0))
+        builder.field(ENABLED_FIELD, enabled)
+        builder.nonOptionalTimeField(LAST_UPDATE_TIME_FIELD, lastUpdateTime)
+        builder.optionalTimeField(ENABLED_TIME_FIELD, enabledTime)
+        builder.field(TRIGGERS_FIELD, triggers.toTypedArray())
         builder.field(Monitor.SCHEMA_VERSION_FIELD, schemaVersion)
         builder.field(QUERY_LANGUAGE_FIELD, queryLanguage.value)
         builder.field(QUERY_FIELD, query)
@@ -331,10 +331,6 @@ data class PPLMonitor(
             requireNotNull(query) { "Query is null" }
             requireNotNull(lastUpdateTime) { "Last update time is null" }
 
-            if (schedule is IntervalSchedule && lookBackWindow != null) {
-                throw IllegalArgumentException("Look back windows only supported for CRON schedules")
-            }
-
             if (queryLanguage == QueryLanguage.SQL) {
                 throw IllegalArgumentException("SQL queries are not supported. Please use a PPL query.")
             }