Github user HyukjinKwon commented on a diff in the pull request: https://github.com/apache/spark/pull/19112#discussion_r136957034 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/streaming/progress.scala --- @@ -200,7 +202,7 @@ class SourceProgress protected[sql]( */ @InterfaceStability.Evolving class SinkProgress protected[sql]( - val description: String) extends Serializable { --- End diff -- I am a committer. Generally, a PR should targets an issue without other changes. I use common sense - fixing styles around the changes is fine if it does not look making revert/backport harder.
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org