Added document _id as param for terms query when searching alerts by … (#753)

stevanbz · web-flow · commit 808494385eb7 · 2023-01-13T11:31:22.000-08:00
* Added document _id as param for terms query when searching alerts by their ids

Signed-off-by: Stevan Buzejic &lt;buzejic.stevan@gmail.com&gt;

* Empty-Commit

Signed-off-by: Stevan Buzejic &lt;buzejic.stevan@gmail.com&gt;

Signed-off-by: Stevan Buzejic &lt;buzejic.stevan@gmail.com&gt;
diff --git a/alerting/src/main/kotlin/org/opensearch/alerting/transport/TransportGetAlertsAction.kt b/alerting/src/main/kotlin/org/opensearch/alerting/transport/TransportGetAlertsAction.kt
@@ -100,7 +100,7 @@ class TransportGetAlertsAction @Inject constructor(
             queryBuilder.filter(QueryBuilders.termQuery("state", getAlertsRequest.alertState))
 
         if (getAlertsRequest.alertIds.isNullOrEmpty() == false) {
-            queryBuilder.filter(QueryBuilders.termsQuery("id", getAlertsRequest.alertIds))
+            queryBuilder.filter(QueryBuilders.termsQuery("_id", getAlertsRequest.alertIds))
         }
 
         if (getAlertsRequest.monitorId != null) {

Original file line number	Diff line number	Diff line change
`@@ -100,7 +100,7 @@ class TransportGetAlertsAction @Inject constructor(`
`100`	`100`	`queryBuilder.filter(QueryBuilders.termQuery("state", getAlertsRequest.alertState))`
`101`	`101`
`102`	`102`	`if (getAlertsRequest.alertIds.isNullOrEmpty() == false) {`
`103`		`- queryBuilder.filter(QueryBuilders.termsQuery("id", getAlertsRequest.alertIds))`
	`103`	`+ queryBuilder.filter(QueryBuilders.termsQuery("_id", getAlertsRequest.alertIds))`
`104`	`104`	`}`
`105`	`105`
`106`	`106`	`if (getAlertsRequest.monitorId != null) {`