Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[SPARK-45640][SQL][TESTS] Fix flaky ProtobufCatalystDataConversionSuite
### What changes were proposed in this pull request? The pr aims to fix flaky ProtobufCatalystDataConversionSuite. As can be seen from the GA test below https://github.com/panbingkun/spark/actions/runs/6612141762/job/17982780917 <img width="988" alt="image" src="https://github.com/apache/spark/assets/15246973/fd1931ba-d858-4c59-9d5c-3a1353e3e005"> When `data.get(0)` is null, `data.get(0).asInstanceOf[Array[Byte]].isEmpty` will be thrown `java.lang.NullPointerException`. <img width="491" alt="image" src="https://github.com/apache/spark/assets/15246973/4cf83e36-1c4a-4b65-b119-1152b026ed29"> ### Why are the changes needed? Fix flaky ProtobufCatalystDataConversionSuite. ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Pass GA. Manually test: ``` ./build/sbt "protobuf/testOnly org.apache.spark.sql.protobuf.ProtobufCatalystDataConversionSuite" ``` <img width="763" alt="image" src="https://github.com/apache/spark/assets/15246973/95e76b2e-5b09-4268-81bf-df20dd262b66"> ### Was this patch authored or co-authored using generative AI tooling? No. Closes #43493 from panbingkun/SPARK-45640. Authored-by: panbingkun <pbk1982@gmail.com> Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
- Loading branch information