[GitHub] spark issue #17903: [SPARK-20661][SparkR][Test] SparkR tableNames() test fai...
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 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 enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #17903: [SPARK-20661][SparkR][Test] SparkR tableNames() test fai...
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 reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #17903: [SPARK-20661][SparkR][Test] SparkR tableNames() test fai...
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 email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #17903: [SPARK-20661][SparkR][Test] SparkR tableNames() test fai...
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 feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #17903: [SPARK-20661][SparkR][Test] SparkR tableNames() test fai...
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 and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #17903: [SPARK-20661][SparkR][Test] SparkR tableNames() test fai...
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. --- 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 enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #17903: [SPARK-20661][SparkR][Test] SparkR tableNames() test fai...
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 enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #17903: [SPARK-20661][SparkR][Test] SparkR tableNames() test fai...
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 [`7020372`](https://github.com/apache/spark/commit/7020372a11aca3a2583cf4f1a36a8da98ad5b482). * This patch passes all tests. * This patch merges cleanly. * This patch adds no public classes. --- 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 enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #17903: [SPARK-20661][SparkR][Test] SparkR tableNames() test fai...
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 before that does not have the failure: https://amplab.cs.berkeley.edu/jenkins/view/Spark%20QA%20Test/job/spark-master-test-sbt-hadoop-2.7/2843/consoleFull --- 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 enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #17903: [SPARK-20661][SparkR][Test] SparkR tableNames() test fai...
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 as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #17903: [SPARK-20661][SparkR][Test] SparkR tableNames() test fai...
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 jenkins. --- 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 enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #17903: [SPARK-20661][SparkR][Test] SparkR tableNames() test fai...
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 [`7020372`](https://github.com/apache/spark/commit/7020372a11aca3a2583cf4f1a36a8da98ad5b482). --- 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 enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org