[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/20795 Thanks! Merged to master --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20795 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92954/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20795 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20795 **[Test build #92954 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92954/testReport)** for PR 20795 at commit [`26f2f54`](https://github.com/apache/spark/commit/26f2f540d30f2e87405489513220468e7708742b). * This patch passes all tests. * This patch merges cleanly. * This patch adds no public classes. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/20795 LGTM --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20795 **[Test build #92954 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92954/testReport)** for PR 20795 at commit [`26f2f54`](https://github.com/apache/spark/commit/26f2f540d30f2e87405489513220468e7708742b). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20795 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92919/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20795 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20795 **[Test build #92919 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92919/testReport)** for PR 20795 at commit [`0db2826`](https://github.com/apache/spark/commit/0db2826e4a7fb6a9e3f435daa33d8cd2080c7929). * This patch passes all tests. * This patch merges cleanly. * This patch adds the following public classes _(experimental)_: * `class CustomerFunctionRegistry extends SimpleFunctionRegistry ` --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20795 **[Test build #92919 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92919/testReport)** for PR 20795 at commit [`0db2826`](https://github.com/apache/spark/commit/0db2826e4a7fb6a9e3f435daa33d8cd2080c7929). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user dilipbiswal commented on the issue: https://github.com/apache/spark/pull/20795 retest this please --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20795 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92915/ Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20795 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20795 **[Test build #92915 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92915/testReport)** for PR 20795 at commit [`0db2826`](https://github.com/apache/spark/commit/0db2826e4a7fb6a9e3f435daa33d8cd2080c7929). * This patch **fails due to an unknown error code, -9**. * This patch merges cleanly. * This patch adds the following public classes _(experimental)_: * `class CustomerFunctionRegistry extends SimpleFunctionRegistry ` --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20795 **[Test build #92915 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92915/testReport)** for PR 20795 at commit [`0db2826`](https://github.com/apache/spark/commit/0db2826e4a7fb6a9e3f435daa33d8cd2080c7929). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20795 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20795 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92849/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20795 **[Test build #92849 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92849/testReport)** for PR 20795 at commit [`8dceda9`](https://github.com/apache/spark/commit/8dceda9cb193aeeddfe34d721b55ea330e1f75be). * This patch passes all tests. * This patch merges cleanly. * This patch adds no public classes. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20795 **[Test build #92849 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92849/testReport)** for PR 20795 at commit [`8dceda9`](https://github.com/apache/spark/commit/8dceda9cb193aeeddfe34d721b55ea330e1f75be). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user kevinyu98 commented on the issue: https://github.com/apache/spark/pull/20795 @gatorsmile Hi Sean, I am so sorry for the long delay. I will address the comments today and submit the code for reviewing. Thanks very much ! Kevin --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/20795 ping @kevinyu98 @dilipbiswal --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20795 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20795 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92333/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20795 **[Test build #92333 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92333/testReport)** for PR 20795 at commit [`d1ee9cb`](https://github.com/apache/spark/commit/d1ee9cb89b9a718fcedea22f19bd7661709fb5fd). * This patch passes all tests. * This patch merges cleanly. * This patch adds no public classes. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20795 **[Test build #92333 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92333/testReport)** for PR 20795 at commit [`d1ee9cb`](https://github.com/apache/spark/commit/d1ee9cb89b9a718fcedea22f19bd7661709fb5fd). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/20795 ok to test --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user kevinyu98 commented on the issue: https://github.com/apache/spark/pull/20795 sorry for the delay. I was working on some other projects. I am back and focus on addressing the comments now. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user maropu commented on the issue: https://github.com/apache/spark/pull/20795 kindly ping @kevinyu98 @dilipbiswal --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20795 Can one of the admins verify this patch? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/20795 ping @kevinyu98 @dilipbiswal --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/20795 Sorry for the late reply. The code looks good to me. The remaining part is to improve the test coverage. Thanks for your great work! --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user kevinyu98 commented on the issue: https://github.com/apache/spark/pull/20795 @gatorsmile Hello Sean, I have updated the code, can you review the code and see anything I need to make changes when you have time? thanks. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20795 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20795 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89142/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20795: [SPARK-23486]cache the function name from the external c...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20795 **[Test build #89142 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89142/testReport)** for PR 20795 at commit [`d1ee9cb`](https://github.com/apache/spark/commit/d1ee9cb89b9a718fcedea22f19bd7661709fb5fd). * This patch passes all tests. * This patch merges cleanly. * This patch adds no public classes. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org