HyukjinKwon commented on code in PR #37263:
URL: https://github.com/apache/spark/pull/37263#discussion_r928377260
##########
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/parquet/ParquetWrite.scala:
##########
@@ -88,6 +89,15 @@ case class ParquetWrite(
// Sets compression scheme
conf.set(ParquetOutputFormat.COMPRESSION,
parquetOptions.compressionCodecClassName)
+ // Sets Zstd level and Zstd workers, only effect when compression codec is
`zstd`
Review Comment:
For clarification, we're already able to set this as a Hadoop property and
this PR exposes it as an option under Spark namespace? We shouldn't probably
overwrites what users specified (by default) here if that's the case.
--
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]
