Removed redundant code.

Signed-off-by: AWSHurneyt <[email protected]>

Author: AWSHurneyt

alerting/src/main/kotlin/org/opensearch/alerting/transport/TransportSearchMonitorAction.kt

@@ -22,6 +22,7 @@ import org.opensearch.cluster.service.ClusterService
 import org.opensearch.common.inject.Inject
 import org.opensearch.common.settings.Settings
 import org.opensearch.commons.alerting.model.Monitor
+import org.opensearch.commons.alerting.model.ScheduledJob
 import org.opensearch.commons.authuser.User
 import org.opensearch.index.query.BoolQueryBuilder
 import org.opensearch.index.query.ExistsQueryBuilder
@@ -53,12 +54,14 @@ class TransportSearchMonitorAction @Inject constructor(
 
     override fun doExecute(task: Task, searchMonitorRequest: SearchMonitorRequest, actionListener: ActionListener<SearchResponse>) {
         val searchSourceBuilder = searchMonitorRequest.searchRequest.source()
-        val queryBuilder = if (searchSourceBuilder.query() == null) BoolQueryBuilder()
-        else QueryBuilders.boolQuery().must(searchSourceBuilder.query())
+            .seqNoAndPrimaryTerm(true)
+            .version(true)
+        val queryBuilder = QueryBuilders.boolQuery().must(searchSourceBuilder.query())
 
-        if (searchMonitorRequest.searchRequest.indices().size == 1 &&
-            !searchMonitorRequest.searchRequest.indices().contains(ALL_ALERT_INDEX_PATTERN)
-        ) {
+        // The SearchMonitor API supports one 'index' parameter of either the SCHEDULED_JOBS_INDEX or ALL_ALERT_INDEX_PATTERN.
+        // When querying the ALL_ALERT_INDEX_PATTERN, we don't want to check whether the MONITOR_TYPE field exists
+        // because we're querying alert indexes.
+        if (searchMonitorRequest.searchRequest.indices().contains(ScheduledJob.SCHEDULED_JOBS_INDEX)) {
             queryBuilder.filter(QueryBuilders.existsQuery(Monitor.MONITOR_TYPE))
         }