You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@spark.apache.org by "Tony Zhang (Jira)" <ji...@apache.org> on 2021/07/16 22:32:00 UTC
[jira] [Created] (SPARK-36187) Commit collision avoidance in
dynamicPartitionOverwrite for non-Parquet formats
Tony Zhang created SPARK-36187:
----------------------------------
Summary: Commit collision avoidance in dynamicPartitionOverwrite for non-Parquet formats
Key: SPARK-36187
URL: https://issues.apache.org/jira/browse/SPARK-36187
Project: Spark
Issue Type: Question
Components: SQL
Affects Versions: 3.1.2
Reporter: Tony Zhang
Hi, my question here is specifically about [PR #29000|https://github.com/apache/spark/pull/29000/files#r649580767] for SPARK-29302.
To my understanding, the PR is to introduce a different staging directory at job commit to avoid commit collision. In SQLHadoopMapReduceCommitProtocol, the new staging directory is only set when SQLConf.OUTPUT_COMMITTER_CLASS is not null: [code|https://github.com/apache/spark/blob/master/sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/SQLHadoopMapReduceCommitProtocol.scala#L58], however in current Spark repo, OUTPUT_COMMITTER_CLASS seems set only for parquet formats: [code|https://github.com/apache/spark/blob/master/sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFileFormat.scala#L96].
However I didn't find similar behavior in Orc related code. Does it mean that this new staging directory will not take effect for non-Parquet formats? Could that be a potential problem? or am I missing something here?
Thanks!
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@spark.apache.org
For additional commands, e-mail: issues-help@spark.apache.org