Github user rednaxelafx commented on the issue:
https://github.com/apache/spark/pull/21039
I just checked the same test in Build 4695, which still has this change,
and the test passed:
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21039
shall we just don't do the nulling out? It wouldn't help the GC a lot.
---
-
To unsubscribe, e-mail:
Github user rednaxelafx commented on the issue:
https://github.com/apache/spark/pull/21039
Thanks for reverting it for me. The test failure was definitely related to
the explicit nulling from this PR, but I can't see how that's possible yet.
First of all, in the build that
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21039
Reverted the PR due to the test failure:
https://amplab.cs.berkeley.edu/jenkins/job/spark-master-test-maven-hadoop-2.7/4694/testReport/junit/org.apache.spark.sql/TPCDSQuerySuite/q61/
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21039
LGTM, merging to master!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21039
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89185/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21039
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21039
**[Test build #89185 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89185/testReport)**
for PR 21039 at commit
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/21039
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21039
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21039
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2201/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21039
**[Test build #89185 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89185/testReport)**
for PR 21039 at commit
12 matches
Mail list logo