anishshri-db opened a new pull request, #43832: URL: https://github.com/apache/spark/pull/43832
### What changes were proposed in this pull request? Fix use of deprecated FileUtils write to pass default charset in RocksDBSuite ### Why are the changes needed? Without the change, we were getting this compilation warning ``` [warn] /Users/anish.shrigondekar/spark/spark/sql/core/src/test/scala/org/apache/spark/sql/execution/streaming/state/RocksDBSuite.scala:854:17: method write in class FileUtils is deprecated [warn] Applicable -Wconf / @nowarn filters for this warning: msg=<part of the message>, cat=deprecation, site=org.apache.spark.sql.execution.streaming.state.RocksDBSuite, origin=org.apache.commons.io.FileUtils.write [warn] FileUtils.write(file2, s"v2\n$json2") [warn] ^ [warn] /Users/anish.shrigondekar/spark/spark/sql/core/src/test/scala/org/apache/spark/sql/execution/streaming/state/RocksDBSuite.scala:1272:17: method write in class FileUtils is deprecated [warn] Applicable -Wconf / @nowarn filters for this warning: msg=<part of the message>, cat=deprecation, site=org.apache.spark.sql.execution.streaming.state.RocksDBSuite.generateFiles.$anonfun, origin=org.apache.commons.io.FileUtils.write [warn] FileUtils.write(file, "a" * length) [warn] ``` ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? Ran test suite ``` 22:47:45.700 WARN org.apache.spark.sql.execution.streaming.state.RocksDBSuite: ===== POSSIBLE THREAD LEAK IN SUITE o.a.s.sql.execution.streaming.state.RocksDBSuite, threads: ForkJoinPool.commonPool-worker-6 (daemon=true), ForkJoinPool.commonPool-worker-4 (daemon=true), rpc-boss-3-1 (daemon=true), ForkJoinPool.commonPool-worker-5 (daemon=true), ForkJoinPool.commonPool-worker-3 (daemon=true), ForkJoinPool.commonPool-worker-2 (daemon=true), shuffle-boss-6-1 (daemon=true), ForkJoinPool.commonPool-worker-1 (daemon=true) ===== [info] Run completed in 1 minute, 55 seconds. [info] Total number of tests run: 77 [info] Suites: completed 1, aborted 0 [info] Tests: succeeded 77, failed 0, canceled 0, ignored 0, pending 0 [info] All tests passed. [success] Total time: 172 s (02:52), completed Nov 15, 2023, 10:47:46 PM ``` ### Was this patch authored or co-authored using generative AI tooling? No -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
