[GitHub] spark issue #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user poplav commented on the issue: https://github.com/apache/spark/pull/17087 Hi, All. I am trying to get this included into Spark 2.1.1. I opened a PR https://github.com/apache/spark/pull/18942. --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/17087 Hi, All. I'm wondering if it is too late for Spark 2.2.0 to include this. Is this too risky for that? --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user zsxwing commented on the issue: https://github.com/apache/spark/pull/17087 LGTM. Thanks! Merging to master. --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76972/ 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #76972 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76972/testReport)** for PR 17087 at commit [`a5fd465`](https://github.com/apache/spark/commit/a5fd4653af29b6549221e6897957cc18c3f3e7df). * 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #76972 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76972/testReport)** for PR 17087 at commit [`a5fd465`](https://github.com/apache/spark/commit/a5fd4653af29b6549221e6897957cc18c3f3e7df). --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76949/ 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #76949 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76949/testReport)** for PR 17087 at commit [`3868bf5`](https://github.com/apache/spark/commit/3868bf5591207f5a1973a17194caa3e7823ef399). * 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #76949 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76949/testReport)** for PR 17087 at commit [`3868bf5`](https://github.com/apache/spark/commit/3868bf5591207f5a1973a17194caa3e7823ef399). --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76862/ 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #76862 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76862/testReport)** for PR 17087 at commit [`1f19c80`](https://github.com/apache/spark/commit/1f19c80391879cdd9a449faa214303b6ac2729da). * This patch passes all tests. * This patch merges cleanly. * This patch adds the following public classes _(experimental)_: * `case class InterpretedPredicate(expression: Expression) extends BasePredicate ` --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #76862 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76862/testReport)** for PR 17087 at commit [`1f19c80`](https://github.com/apache/spark/commit/1f19c80391879cdd9a449faa214303b6ac2729da). --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user kiszk commented on the issue: https://github.com/apache/spark/pull/17087 ping @marmbrus --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user kiszk commented on the issue: https://github.com/apache/spark/pull/17087 @marmbrus could you please take a look? --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75999/ 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #75999 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75999/testReport)** for PR 17087 at commit [`8b6ba75`](https://github.com/apache/spark/commit/8b6ba755f75fc678b732b408e17c4a4670da1d92). * 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #75999 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75999/testReport)** for PR 17087 at commit [`8b6ba75`](https://github.com/apache/spark/commit/8b6ba755f75fc678b732b408e17c4a4670da1d92). --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75961/ 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #75961 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75961/testReport)** for PR 17087 at commit [`c2e6b8c`](https://github.com/apache/spark/commit/c2e6b8cabc820d24e882d3441d1738ab0368cecb). * 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #75961 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75961/testReport)** for PR 17087 at commit [`c2e6b8c`](https://github.com/apache/spark/commit/c2e6b8cabc820d24e882d3441d1738ab0368cecb). --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75951/ 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #75951 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75951/testReport)** for PR 17087 at commit [`ea67c8a`](https://github.com/apache/spark/commit/ea67c8a423f760f30fae76c4fe85c275a3800564). * 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #75951 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75951/testReport)** for PR 17087 at commit [`ea67c8a`](https://github.com/apache/spark/commit/ea67c8a423f760f30fae76c4fe85c275a3800564). --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #75950 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75950/testReport)** for PR 17087 at commit [`51a71a6`](https://github.com/apache/spark/commit/51a71a6fc0a611cf04e9a77fe13a3f3606548ec1). * This patch **fails Scala style 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75950/ Test FAILed. --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 Merged build finished. Test FAILed. --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #75950 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75950/testReport)** for PR 17087 at commit [`51a71a6`](https://github.com/apache/spark/commit/51a71a6fc0a611cf04e9a77fe13a3f3606548ec1). --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #75948 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75948/testReport)** for PR 17087 at commit [`8548b0e`](https://github.com/apache/spark/commit/8548b0e93ed036bc1fcd1c9a259cf82cc80dca48). * This patch **fails Scala style 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75948/ Test FAILed. --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 Merged build finished. Test FAILed. --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #75948 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75948/testReport)** for PR 17087 at commit [`8548b0e`](https://github.com/apache/spark/commit/8548b0e93ed036bc1fcd1c9a259cf82cc80dca48). --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user kiszk commented on the issue: https://github.com/apache/spark/pull/17087 @marmbrus could you please take a look? --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75487/ 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #75487 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75487/testReport)** for PR 17087 at commit [`81545d6`](https://github.com/apache/spark/commit/81545d6b02bf90d0d546ac85396dcfda474de985). * 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #75487 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75487/testReport)** for PR 17087 at commit [`81545d6`](https://github.com/apache/spark/commit/81545d6b02bf90d0d546ac85396dcfda474de985). --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user kiszk commented on the issue: https://github.com/apache/spark/pull/17087 @marmbrus could you please take a look? --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user kiszk commented on the issue: https://github.com/apache/spark/pull/17087 ping @marmbrus --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user kiszk commented on the issue: https://github.com/apache/spark/pull/17087 ping @marmbrus, @davies --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user kiszk commented on the issue: https://github.com/apache/spark/pull/17087 ping @marmbrus, @davies --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user kiszk commented on the issue: https://github.com/apache/spark/pull/17087 @marmbrus, could you please take a look at this PR for fixing a problem? --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user kiszk commented on the issue: https://github.com/apache/spark/pull/17087 ping @marmbrus --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74464/ 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #74464 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74464/testReport)** for PR 17087 at commit [`7db9aea`](https://github.com/apache/spark/commit/7db9aea707ee5c5949ffd61ac32a49518f144dc1). * 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #74464 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74464/testReport)** for PR 17087 at commit [`7db9aea`](https://github.com/apache/spark/commit/7db9aea707ee5c5949ffd61ac32a49518f144dc1). --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 Merged build finished. Test FAILed. --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74442/ Test FAILed. --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #74442 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74442/testReport)** for PR 17087 at commit [`530f84f`](https://github.com/apache/spark/commit/530f84fae02659a914009fb49f42ec880f13301c). * This patch **fails Spark unit tests**. * This patch merges cleanly. * This patch adds the following public classes _(experimental)_: * `class InterpretedPredicate(expression: Expression) extends BasePredicate ` --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74435/ Test FAILed. --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 Merged build finished. Test FAILed. --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #74442 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74442/testReport)** for PR 17087 at commit [`530f84f`](https://github.com/apache/spark/commit/530f84fae02659a914009fb49f42ec880f13301c). --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #74435 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74435/testReport)** for PR 17087 at commit [`11f56d1`](https://github.com/apache/spark/commit/11f56d185228a32ead8cb7d1ccb3e243b6a9383b). * This patch **fails Spark unit 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #74435 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74435/testReport)** for PR 17087 at commit [`11f56d1`](https://github.com/apache/spark/commit/11f56d185228a32ead8cb7d1ccb3e243b6a9383b). --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user kiszk commented on the issue: https://github.com/apache/spark/pull/17087 Jenkins, retest this please --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 Merged build finished. Test FAILed. --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74428/ Test FAILed. --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #74428 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74428/testReport)** for PR 17087 at commit [`11f56d1`](https://github.com/apache/spark/commit/11f56d185228a32ead8cb7d1ccb3e243b6a9383b). --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74374/ 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #74374 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74374/testReport)** for PR 17087 at commit [`c5fc5f1`](https://github.com/apache/spark/commit/c5fc5f140a9c5d661748033f4bad0f59e1ca88bb). * This patch passes all tests. * This patch merges cleanly. * This patch adds the following public classes _(experimental)_: * `class InterpretedPredicate(expression: Expression) extends GenPredicate ` --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #74374 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74374/testReport)** for PR 17087 at commit [`c5fc5f1`](https://github.com/apache/spark/commit/c5fc5f140a9c5d661748033f4bad0f59e1ca88bb). --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74313/ 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #74313 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74313/testReport)** for PR 17087 at commit [`a2f85cd`](https://github.com/apache/spark/commit/a2f85cd7d529ff48128710a1aa1dcf760025080d). * This patch passes all tests. * This patch merges cleanly. * This patch adds the following public classes _(experimental)_: * `class InterpretedPredicate(expression: Expression) extends GenPredicate ` --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #74313 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74313/testReport)** for PR 17087 at commit [`a2f85cd`](https://github.com/apache/spark/commit/a2f85cd7d529ff48128710a1aa1dcf760025080d). --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74311/ Test FAILed. --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #74311 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74311/testReport)** for PR 17087 at commit [`c02589c`](https://github.com/apache/spark/commit/c02589c9343809b4dea2e45cc1b45c8519b1fefa). * This patch **fails to build**. * 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 Merged build finished. Test FAILed. --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 Build finished. Test FAILed. --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #74310 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74310/testReport)** for PR 17087 at commit [`5fb413f`](https://github.com/apache/spark/commit/5fb413fc53a173665ff22bb893f6a6143602d602). * This patch **fails to build**. * This patch **does not merge cleanly**. * This patch adds the following public classes _(experimental)_: * `class InterpretedPredicate(expression: Expression) extends GenPredicate ` --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74310/ Test FAILed. --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #74311 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74311/testReport)** for PR 17087 at commit [`c02589c`](https://github.com/apache/spark/commit/c02589c9343809b4dea2e45cc1b45c8519b1fefa). --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user kiszk commented on the issue: https://github.com/apache/spark/pull/17087 Thank you for pointing out `InterpretedPredicate`. Now, `newPredicate` always returns `Predicate` that can be executed by calling `eval()`. It looks simpler and better than the first implementation. --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #74310 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74310/testReport)** for PR 17087 at commit [`5fb413f`](https://github.com/apache/spark/commit/5fb413fc53a173665ff22bb893f6a6143602d602). --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user marmbrus commented on the issue: https://github.com/apache/spark/pull/17087 There appears to have been some code drift (as `GeneratePredicate` and `InterpretedPredicate` both used to return a class that inherited from a common interface), but I don't think its hard to just isolate the fault handling in `newPredicate`. Just fix [InterpretedPredicate](https://github.com/apache/spark/blob/9b7a03f15ac45e5f7dcf118d1e7ce1556339aa46/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala) to actually return a `Predicate` rather than a bare lambda function. The code there already handles binding and evaluation. --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74262/ 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #74262 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74262/testReport)** for PR 17087 at commit [`0e2bbe7`](https://github.com/apache/spark/commit/0e2bbe7f4b27c5b6952d695eaca24c171fb6c861). * 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user kiszk commented on the issue: https://github.com/apache/spark/pull/17087 @marmbrus I have just commit the code of intermediate refactoring. Would it be possible to give comments? If it is fine (return `null` and check it at caller), I will update other caller sites for `newPredicate`. --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #74262 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74262/testReport)** for PR 17087 at commit [`0e2bbe7`](https://github.com/apache/spark/commit/0e2bbe7f4b27c5b6952d695eaca24c171fb6c861). --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user kiszk commented on the issue: https://github.com/apache/spark/pull/17087 I am refactoring `newPredicate`. `newPredicate` will catch the exception and log a warning. However, I think that `newPredicate` cannot return an interpreted result. This is because code generation return `Predicate` while ` BindReferences.bindReference(condition, child.output)` returns `Expression`. These two classes exists in different class hierarchy. --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user marmbrus commented on the issue: https://github.com/apache/spark/pull/17087 I don't think we need a complex refactoring. Why can't `newPredicate` catch the exception, log a warning and return an interpreted `Predicate`? --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user kiszk commented on the issue: https://github.com/apache/spark/pull/17087 @marmbrus thank you for your comments. For feedback mechanism, I imagine that you are talking about [this](https://github.com/apache/spark/blob/master/sql/core/src/main/scala/org/apache/spark/sql/execution/WholeStageCodegenExec.scala#L357-L364). When the whole-stage codegen is enabled, this fallback works. In this case, the whole-stage codegen is disabled since the number of fields are big and [these two variables](https://github.com/apache/spark/blob/master/sql/core/src/main/scala/org/apache/spark/sql/execution/WholeStageCodegenExec.scala#L463-L466) will have `false`. I agree that we should prepare more generic approach that can be applicable to more cases. However, it is not easy to implement for now. It would require some refactoring. As one of simple refactoring, if an exception occurs, `newPredicate` returns `null` or `none`. Then, caller of `newPredicate` always checks return value for handling of code generation failure. It make caller code simpler. While more complex refactoring could be applicable, another PR should address it. What do you think? FYI: I have just noticed that this problem may occur at [CartesianProductExec](https://github.com/apache/spark/blob/master/sql/core/src/main/scala/org/apache/spark/sql/execution/joins/CartesianProductExec.scala#L104-L110) and [InMemoryTableScanExec](https://github.com/apache/spark/blob/master/sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryTableScanExec.scala#L136). I have to do the same thing in these two places. --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user marmbrus commented on the issue: https://github.com/apache/spark/pull/17087 I agree with the general approach of having a fallback from code generation to interpreted evaluation, but I also agree that this feels too narrowly targeted. In particular, why do this in one operator rather than in `newPredicate` (or maybe even in codegen itself). Another thing that maybe @davies can comment on, I thought we already had this fallback implemented? So I'm curious why its not already handling this test case. Maybe there is an existing mechanism we just need to make more general. --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74019/ 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17087 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #74019 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74019/testReport)** for PR 17087 at commit [`98cd961`](https://github.com/apache/spark/commit/98cd961fa8602c981ff24de76ebd480b6f177429). * 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #74019 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74019/testReport)** for PR 17087 at commit [`98cd961`](https://github.com/apache/spark/commit/98cd961fa8602c981ff24de76ebd480b6f177429). --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user kiszk commented on the issue: https://github.com/apache/spark/pull/17087 I already identified where 64k limit is exceeded. As a result, I think that it is not easy to fix the issue. The previous issues are derived from independent blocks (in `Seq[ExprCode]`) that can be splited into multiple methods. On the other hand, it is not easy split a tree with many nodes (in `ExprCode`) that cannot be into multiple methods. It is very hard to split a `ExprCode` into multiple correct `ExprCode`s. For catching exception, you are right. I have to catch only an exception related to compilation errors. I will address it by handling nested exceptions. I will address 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user srowen commented on the issue: https://github.com/apache/spark/pull/17087 Is it right to just do this in one code path? why not all similar cases where the 64k limit is exceeded? Can the underlying problem be fixed or improved? You're also catching all Exceptions, why not something more specific? is it possible? It feels like a funny change to make for these reasons. --- 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user kiszk commented on the issue: https://github.com/apache/spark/pull/17087 @davies, could you please review 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #73530 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73530/testReport)** for PR 17087 at commit [`6f40a93`](https://github.com/apache/spark/commit/6f40a93cfb21597b214f930d3a5bd9c6645ef227). * 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 #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17087 **[Test build #73530 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73530/testReport)** for PR 17087 at commit [`6f40a93`](https://github.com/apache/spark/commit/6f40a93cfb21597b214f930d3a5bd9c6645ef227). --- 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
