[SPARK-51535][SQL] Don't compare hash codes directly in ProtobufCatalystDataConversionSuite

vladimirg-db · HyukjinKwon · commit ecc659560597 · 2025-03-18T08:37:24.000+09:00
### What changes were proposed in this pull request? Don't compare hash codes directly in `ProtobufCatalystDataConversionSuite` A followup for #50295 ### Why are the changes needed? `hashCode` implementations are not guaranteed to be the same across different JVMs and platforms. ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? `build/sbt "testOnly org.apache.spark.sql.protobuf.ProtobufCatalystDataConversionSuite -- -z \"hash\""` ### Was this patch authored or co-authored using generative AI tooling? No. Closes #50298 from vladimirg-db/vladimir-golubev_data/fix-protobuf-failed-tests-2. Authored-by: Vladimir Golubev <vladimir.golubev@databricks.com> Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
diff --git a/connector/protobuf/src/test/scala/org/apache/spark/sql/protobuf/ProtobufCatalystDataConversionSuite.scala b/connector/protobuf/src/test/scala/org/apache/spark/sql/protobuf/ProtobufCatalystDataConversionSuite.scala
@@ -380,9 +380,6 @@ class ProtobufCatalystDataConversionSuite
       Map("mode" -> "PERMISSIVE")
     )
 
-    assert(
-      protobufDataToCatalyst.hashCode == -937893175
-    )
     assert(
       protobufDataToCatalyst
         .copy(options = Map("mode" -> "FAILFAST"))

Original file line number	Diff line number	Diff line change
`@@ -380,9 +380,6 @@ class ProtobufCatalystDataConversionSuite`
`380`	`380`	`Map("mode" -> "PERMISSIVE")`
`381`	`381`	`)`
`382`	`382`
`383`		`- assert(`
`384`		`- protobufDataToCatalyst.hashCode == -937893175`
`385`		`- )`
`386`	`383`	`assert(`
`387`	`384`	`protobufDataToCatalyst`
`388`	`385`	`.copy(options = Map("mode" -> "FAILFAST"))`