HyukjinKwon commented on a change in pull request #33148:
URL: https://github.com/apache/spark/pull/33148#discussion_r661117144



##########
File path: 
core/src/main/scala/org/apache/spark/internal/io/FileCommitProtocol.scala
##########
@@ -107,10 +111,9 @@ abstract class FileCommitProtocol extends Logging {
    * if a task is going to write out multiple files to the same dir. The file 
commit protocol only
    * guarantees that files written by different tasks will not conflict.
    *
-   * This API should be implemented and called, instead of
-   * [[newTaskTempFile(taskContest, dir, ext)]]. Provide a default 
implementation here to be
-   * backward compatible with custom [[FileCommitProtocol]] implementations 
before Spark 3.2.0.
+   * @since 3.2.0
    */
+  @Unstable

Review comment:
       @c21 can we mark this whole class as `@Unstable`?




-- 
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]

Reply via email to