AngersZhuuuu commented on a change in pull request #33828:
URL: https://github.com/apache/spark/pull/33828#discussion_r697148671
##########
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala
##########
@@ -1216,6 +1216,16 @@ object SQLConf {
.booleanConf
.createWithDefault(true)
+ val FILE_COMMIT_STAGING_DIR =
Review comment:
> It's reasonable, but it's an assumption based on this PR AS-IS scope.
Some other PRs may try to use it later.
I'm planning to optimize the Commit protocol too and discuss with @cloud-fan
https://docs.google.com/document/d/13yzpIUAmgQaJ1Jnu0kqQ4DORDxQoZJmWaJMAVMajdi0/edit#
If we can do like that, I think it's will be more convenient for us to do
more optimize on sql part
--
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]
