misc review-based cleanup

Signed-off-by: Dennis Toepker <[email protected]>
opensearch-project · Jun 6, 2024 · 9ad5a15 · 9ad5a15
1 parent 3b3566e
commit 9ad5a15
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 8 deletions.
diff --git a/src/main/kotlin/org/opensearch/commons/alerting/action/AlertingActions.kt b/src/main/kotlin/org/opensearch/commons/alerting/action/AlertingActions.kt
@@ -21,9 +21,9 @@ object AlertingActions {
     const val SUBSCRIBE_FINDINGS_ACTION_NAME = "cluster:admin/opensearch/alerting/findings/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/opendistro/alerting/alerts/comments/write"
-    const val SEARCH_COMMENTS_ACTION_NAME = "cluster:admin/opendistro/alerting/alerts/comments/search"
-    const val DELETE_COMMENT_ACTION_NAME = "cluster:admin/opendistro/alerting/alerts/comments/delete"
+    const val INDEX_COMMENT_ACTION_NAME = "cluster:admin/opensearch/alerting/alerts/comments/write"
+    const val SEARCH_COMMENTS_ACTION_NAME = "cluster:admin/opensearch/alerting/alerts/comments/search"
+    const val DELETE_COMMENT_ACTION_NAME = "cluster:admin/opensearch/alerting/alerts/comments/delete"
 
     @JvmField
     val INDEX_MONITOR_ACTION_TYPE =

diff --git a/src/main/kotlin/org/opensearch/commons/alerting/model/Comment.kt b/src/main/kotlin/org/opensearch/commons/alerting/model/Comment.kt
@@ -32,11 +32,7 @@ data class Comment(
         content = sin.readString(),
         createdTime = sin.readInstant(),
         lastUpdatedTime = sin.readOptionalInstant(),
-        user = if (sin.readBoolean()) {
-            User(sin)
-        } else {
-            null
-        }
+        user = if (sin.readBoolean()) User(sin) else null
     )
 
     constructor(