Revert "add fields param in toxcontent() for doc level query (opensea…

…rch-project#549) (opensearch-project#555)" This reverts commit dcf288d. Signed-off-by: Surya Sashank Nistala <snistala@amazon.com>
eirsep · Nov 27, 2023 · af50b2e · af50b2e
1 parent 92da659
commit af50b2e
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 13 deletions.
diff --git a/src/main/kotlin/org/opensearch/commons/alerting/model/DocLevelQuery.kt b/src/main/kotlin/org/opensearch/commons/alerting/model/DocLevelQuery.kt
@@ -59,7 +59,6 @@ data class DocLevelQuery(
         builder.startObject()
             .field(QUERY_ID_FIELD, id)
             .field(NAME_FIELD, name)
-            .field(FIELDS_FIELD, fields.toTypedArray())
             .field(QUERY_FIELD, query)
             .field(TAGS_FIELD, tags.toTypedArray())
             .endObject()

diff --git a/src/test/kotlin/org/opensearch/commons/alerting/model/XContentTests.kt b/src/test/kotlin/org/opensearch/commons/alerting/model/XContentTests.kt
@@ -417,18 +417,6 @@ class XContentTests {
         )
     }
 
-    @Test
-    fun `test doc level query toXcontent`() {
-        val dlq = DocLevelQuery("id", "name", listOf("f1", "f2"), "query", listOf("t1", "t2"))
-        val dlqString = dlq.toXContent(builder(), ToXContent.EMPTY_PARAMS).string()
-        val parsedDlq = DocLevelQuery.parse(parser(dlqString))
-        Assertions.assertEquals(
-            dlq,
-            parsedDlq,
-            "Round tripping Doc level query doesn't work"
-        )
-    }
-
     @Test
     fun `test alert parsing`() {
         val alert = randomAlert()