SuccessChanges

Summary

  1. [SPARK-24168][SQL] WindowExec should not access SQLConf at executor side (commit: d35eb2f9b0af1a625749ca8b7f12d8eceed28766) (details)
  2. [SPARK-23697][CORE] LegacyAccumulatorWrapper should define isZero (commit: 3f78f60cc80d04ce762bd332dd2fa5f7d17c2b3c) (details)
Commit d35eb2f9b0af1a625749ca8b7f12d8eceed28766 by gatorsmile
[SPARK-24168][SQL] WindowExec should not access SQLConf at executor side
## What changes were proposed in this pull request?
This PR is extracted from #21190 , to make it easier to backport.
`WindowExec#createBoundOrdering` is called on executor side, so we can't
use `conf.sessionLocalTimezone` there.
## How was this patch tested?
tested in #21190
Author: Wenchen Fan <wenchen@databricks.com>
Closes #21225 from cloud-fan/minor3.
(cherry picked from commit e646ae67f2e793204bc819ab2b90815214c2bbf3)
Signed-off-by: gatorsmile <gatorsmile@gmail.com>
(commit: d35eb2f9b0af1a625749ca8b7f12d8eceed28766)
The file was modifiedsql/core/src/main/scala/org/apache/spark/sql/execution/window/WindowExec.scala (diff)
Commit 3f78f60cc80d04ce762bd332dd2fa5f7d17c2b3c by wenchen
[SPARK-23697][CORE] LegacyAccumulatorWrapper should define isZero
correctly
## What changes were proposed in this pull request?
It's possible that Accumulators of Spark 1.x may no longer work with
Spark 2.x. This is because `LegacyAccumulatorWrapper.isZero` may return
wrong answer if `AccumulableParam` doesn't define equals/hashCode.
This PR fixes this by using reference equality check in
`LegacyAccumulatorWrapper.isZero`.
## How was this patch tested?
a new test
Author: Wenchen Fan <wenchen@databricks.com>
Closes #21229 from cloud-fan/accumulator.
(cherry picked from commit 4d5de4d303a773b1c18c350072344bd7efca9fc4)
Signed-off-by: Wenchen Fan <wenchen@databricks.com>
(commit: 3f78f60cc80d04ce762bd332dd2fa5f7d17c2b3c)
The file was modifiedcore/src/test/scala/org/apache/spark/util/AccumulatorV2Suite.scala (diff)
The file was modifiedcore/src/main/scala/org/apache/spark/util/AccumulatorV2.scala (diff)