[8.3] Fix object equals for SqlQueryRequest's binaryCommunication (#87887) (#89143)

* Fix object equals for SqlQueryRequest's binaryCommunication (#87887)

Co-authored-by: owenniceliu <[email protected]>

* fix the changelog yaml file for pr 87887 (#89146)

Co-authored-by: wjwei <[email protected]>
Co-authored-by: owenniceliu <[email protected]>

Author: 3 people

docs/changelog/87887.yaml

@@ -0,0 +1,5 @@
+pr: 87887
+summary: fix object equals
+area: SQL
+type: bug
+issues: []

x-pack/plugin/sql/sql-action/src/main/java/org/elasticsearch/xpack/sql/action/SqlQueryRequest.java

@@ -325,7 +325,7 @@ public boolean equals(Object obj) {
         return super.equals(obj)
             && fieldMultiValueLeniency == ((SqlQueryRequest) obj).fieldMultiValueLeniency
             && indexIncludeFrozen == ((SqlQueryRequest) obj).indexIncludeFrozen
-            && binaryCommunication == ((SqlQueryRequest) obj).binaryCommunication
+            && Objects.equals(binaryCommunication, ((SqlQueryRequest) obj).binaryCommunication)
             && keepOnCompletion == ((SqlQueryRequest) obj).keepOnCompletion
             && allowPartialSearchResults == ((SqlQueryRequest) obj).allowPartialSearchResults
             && Objects.equals(cursor, ((SqlQueryRequest) obj).cursor)