Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/20386
@gengliangwang, what is the use case supported by this? In other words, how
is `onTaskCommit(taskCommit: TaskCommitMessage)` currently used that requires
this change?
In general, I'm more concerned with the batch side and I don't have a huge
problem with this change. I do want to make sure it is in support of a valid
use case. I'd also rather separate the batch and streaming committer APIs
because they have so little in common.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
