Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21341
Yea we may see flaky test for a while. Fortunately #21299 is there for a
while and I think it's close to merge, so it should be fine :)
---
Github user squito commented on the issue:
https://github.com/apache/spark/pull/21341
sure this is fine, but we'll see the flakiness back in the builds till
https://github.com/apache/spark/pull/21299 is merged, right?
---
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21341
thanks, merging to master!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21341
to unblock the improvement PR, I'm going to merge it first and rebase that
PR. I will still keep the driver-side check for `SparkSession`, which is what
@squito concerned before.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21341
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/21341
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90673/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21341
**[Test build #90673 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90673/testReport)**
for PR 21341 at commit
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21341
Personally I feel it should be safe to do the revert since we have a better
approach, but I'd prefer to hear what @squito think about this.
---
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21341
lgtm too
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21341
LGTM too
---
-
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/21341
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/21341
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/3253/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21341
**[Test build #90673 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90673/testReport)**
for PR 21341 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21341
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21341
retest this please.
---
-
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/21341
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90672/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21341
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21341
**[Test build #90672 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90672/testReport)**
for PR 21341 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21341
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/21341
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/3252/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21341
**[Test build #90672 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90672/testReport)**
for PR 21341 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21341
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21341
cc @gatorsmile @viirya @jiangxb1987
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
23 matches
Mail list logo
