[GitHub] [spark] itayB commented on pull request #32518: [SPARK-35383][CORE] Improve s3a magic committer support by inferring missing configs

[GitBox <gi...@apache.org>]

itayB commented on pull request #32518:
URL: https://github.com/apache/spark/pull/32518#issuecomment-1048780380


   Hi @dongjoon-hyun!
   
   I am using this new committer - thanks for that!
   I wonder if those [recommendations](https://spark.apache.org/docs/latest/cloud-integration.html#recommended-settings-for-writing-to-object-stores) are still required/relevant:
   ```
   spark.hadoop.mapreduce.fileoutputcommitter.algorithm.version 2
   ```
   and the parquet [recommendations](https://spark.apache.org/docs/latest/cloud-integration.html#parquet-io-settings):
   ```
   spark.hadoop.parquet.enable.summary-metadata false
   spark.sql.parquet.mergeSchema false
   spark.sql.parquet.filterPushdown true
   spark.sql.hive.metastorePartitionPruning true
   ```
   or should I remove some of those if I use the magic committer.
   
   Thanks


-- 
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: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org