Revert "updating PublishFindingsRequest to use a list of findings rather than… (#832)" (#847)

This reverts commit 774c00d.

Signed-off-by: Dennis Toepker <[email protected]>
Co-authored-by: Dennis Toepker <[email protected]>

Author: toepkerd

src/main/kotlin/org/opensearch/commons/alerting/AlertingPluginInterface.kt

@@ -27,7 +27,6 @@ import org.opensearch.commons.alerting.action.IndexMonitorRequest
 import org.opensearch.commons.alerting.action.IndexMonitorResponse
 import org.opensearch.commons.alerting.action.IndexWorkflowRequest
 import org.opensearch.commons.alerting.action.IndexWorkflowResponse
-import org.opensearch.commons.alerting.action.PublishBatchFindingsRequest
 import org.opensearch.commons.alerting.action.PublishFindingsRequest
 import org.opensearch.commons.alerting.action.SearchMonitorRequest
 import org.opensearch.commons.alerting.action.SubscribeFindingsResponse
@@ -269,24 +268,6 @@ object AlertingPluginInterface {
         )
     }
 
-    fun publishBatchFindings(
-        client: NodeClient,
-        request: PublishBatchFindingsRequest,
-        listener: ActionListener<SubscribeFindingsResponse>
-    ) {
-        client.execute(
-            AlertingActions.SUBSCRIBE_BATCH_FINDINGS_ACTION_TYPE,
-            request,
-            wrapActionListener(listener) { response ->
-                recreateObject(response) {
-                    SubscribeFindingsResponse(
-                        it
-                    )
-                }
-            }
-        )
-    }
-
     /**
      * Acknowledge Chained Alerts interface.
      * @param client Node client for making transport action

src/main/kotlin/org/opensearch/commons/alerting/action/AlertingActions.kt

@@ -19,7 +19,6 @@ object AlertingActions {
     const val ACKNOWLEDGE_ALERTS_ACTION_NAME = "cluster:admin/opendistro/alerting/alerts/ack"
     const val ACKNOWLEDGE_CHAINED_ALERTS_ACTION_NAME = "cluster:admin/opendistro/alerting/chained_alerts/ack"
     const val SUBSCRIBE_FINDINGS_ACTION_NAME = "cluster:admin/opensearch/alerting/findings/subscribe"
-    const val SUBSCRIBE_BATCH_FINDINGS_ACTION_NAME = "cluster:admin/opensearch/alerting/findings/batch/subscribe"
     const val GET_MONITOR_ACTION_NAME = "cluster:admin/opendistro/alerting/monitor/get"
     const val SEARCH_MONITORS_ACTION_NAME = "cluster:admin/opendistro/alerting/monitor/search"
     const val INDEX_COMMENT_ACTION_NAME = "cluster:admin/opensearch/alerting/comments/write"
@@ -66,10 +65,6 @@ object AlertingActions {
     val SUBSCRIBE_FINDINGS_ACTION_TYPE =
         ActionType(SUBSCRIBE_FINDINGS_ACTION_NAME, ::SubscribeFindingsResponse)
 
-    @JvmField
-    val SUBSCRIBE_BATCH_FINDINGS_ACTION_TYPE =
-        ActionType(SUBSCRIBE_BATCH_FINDINGS_ACTION_NAME, ::SubscribeFindingsResponse)
-
     @JvmField
     val ACKNOWLEDGE_CHAINED_ALERTS_ACTION_TYPE =
         ActionType(ACKNOWLEDGE_CHAINED_ALERTS_ACTION_NAME, ::AcknowledgeAlertResponse)