Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/17903
ah ok, i thought we said it was this build introducing this failure
https://github.com/apache/spark/pull/17892#issuecomment-299920986
thanks, it's good to know
---
If your project is
Github user yhuai commented on the issue:
https://github.com/apache/spark/pull/17903
I do not think https://github.com/apache/spark/pull/17649 caused the
problem. I saw failures without that internally.
---
If your project is set up for it, you can reply to this email and have your
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/17903
Hmm, do we know why temp tables are getting leak since
https://github.com/apache/spark/pull/17649
But not before?
---
If your project is set up for it, you can reply to this
Github user yhuai commented on the issue:
https://github.com/apache/spark/pull/17903
Thanks @falaki. Merging to master and branch-2.2.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user yhuai commented on the issue:
https://github.com/apache/spark/pull/17903
Seems 2.2 build is fine. But, I'd like to get this merged in branch-2.2
since this test will fail if any previous tests leak tables.
---
If your project is set up for it, you can reply to this email
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17903
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76592/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17903
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17903
**[Test build #76592 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76592/testReport)**
for PR 17903 at commit
Github user falaki commented on the issue:
https://github.com/apache/spark/pull/17903
@shivaram it seems to have started today with this build:
https://amplab.cs.berkeley.edu/jenkins/view/Spark%20QA%20Test/job/spark-master-test-sbt-hadoop-2.7/2844/consoleFull
The build just
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/17903
LGTM. Thanks @falaki -- BTW is this a problem only on master or should we
also backport this ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user yhuai commented on the issue:
https://github.com/apache/spark/pull/17903
@felixcheung fyi. I think the main problem of this test is that it will be
broken if tests executed before this one leak any table. I think this change
makes sense. I will merge it once it passes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17903
**[Test build #76592 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76592/testReport)**
for PR 17903 at commit
12 matches
Mail list logo