Github user sohama4 commented on the issue:
https://github.com/apache/spark/pull/21316
@kiszk @jaceklaskowski @rxin @zsxwing @maropu can we move forward here?
Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr
Github user sohama4 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21316#discussion_r188143976
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -1607,7 +1607,9 @@ class Dataset[T] private[sql
Github user sohama4 commented on the issue:
https://github.com/apache/spark/pull/21316
Thanks for the approval @jaceklaskowski! Can you leave a comment so that
Jenkins can get testing underway?
---
-
To unsubscribe
Github user sohama4 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21316#discussion_r187832727
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -1607,7 +1607,9 @@ class Dataset[T] private[sql
Github user sohama4 commented on the issue:
https://github.com/apache/spark/pull/21316
@vanzin @maropu @gatorsmile @jaceklaskowski can someone take a look at
this? I'd appreciate it!
---
-
To unsubscribe, e-mail
GitHub user sohama4 opened a pull request:
https://github.com/apache/spark/pull/21316
[SPARK-20538][SQL] Wrap Dataset.reduce with withNewExecutionId.
## What changes were proposed in this pull request?
Wrap Dataset.reduce with `withNewExecutionId`.
You can merge this pull