[GitHub] spark issue #23263: [SPARK-23674][ML] Adds Spark ML Events

2018-12-09 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/23263 > Visualizing a workflow is nice, but Spark's Pipelines are typically pretty straightforward and linear. I could imagine producing a nicer visualization than what you get from reading the Spark

[GitHub] spark issue #23263: [SPARK-23674][ML] Adds Spark ML Events

2018-12-09 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/23263 My first impression is that it's a big change, which is reason for caution here. Visualizing a workflow is nice, but Spark's Pipelines are typically pretty straightforward and linear. I

[GitHub] spark issue #23263: [SPARK-23674][ML] Adds Spark ML Events

2018-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/23263 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #23263: [SPARK-23674][ML] Adds Spark ML Events

2018-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/23263 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99873/ Test PASSed. ---

[GitHub] spark issue #23263: [SPARK-23674][ML] Adds Spark ML Events

2018-12-08 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/23263 **[Test build #99873 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99873/testReport)** for PR 23263 at commit

[GitHub] spark issue #23263: [SPARK-23674][ML] Adds Spark ML Events

2018-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/23263 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99872/ Test PASSed. ---

[GitHub] spark issue #23263: [SPARK-23674][ML] Adds Spark ML Events

2018-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/23263 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #23263: [SPARK-23674][ML] Adds Spark ML Events

2018-12-08 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/23263 **[Test build #99872 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99872/testReport)** for PR 23263 at commit

[GitHub] spark issue #23263: [SPARK-23674][ML] Adds Spark ML Events

2018-12-08 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/23263 **[Test build #99873 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99873/testReport)** for PR 23263 at commit

[GitHub] spark issue #23263: [SPARK-23674][ML] Adds Spark ML Events

2018-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/23263 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark issue #23263: [SPARK-23674][ML] Adds Spark ML Events

2018-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/23263 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #23263: [SPARK-23674][ML] Adds Spark ML Events

2018-12-08 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/23263 **[Test build #99872 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99872/testReport)** for PR 23263 at commit

[GitHub] spark issue #23263: [SPARK-23674][ML] Adds Spark ML Events

2018-12-08 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/23263 The tests pass in my local. I'll fix them shortly. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For

[GitHub] spark issue #23263: [SPARK-23674][ML] Adds Spark ML Events

2018-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/23263 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #23263: [SPARK-23674][ML] Adds Spark ML Events

2018-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/23263 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99868/ Test FAILed. ---

[GitHub] spark issue #23263: [SPARK-23674][ML] Adds Spark ML Events

2018-12-08 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/23263 **[Test build #99868 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99868/testReport)** for PR 23263 at commit

[GitHub] spark issue #23263: [SPARK-23674][ML] Adds Spark ML Events

2018-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/23263 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark issue #23263: [SPARK-23674][ML] Adds Spark ML Events

2018-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/23263 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #23263: [SPARK-23674][ML] Adds Spark ML Events

2018-12-08 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/23263 cc @srowen, @cloud-fan (since it mimics SQL's event listener), @jkbradley, @mengxr and @yanboliang. Mind if I ask to take a look please? WDYT about this? ---

[GitHub] spark issue #23263: [SPARK-23674][ML] Adds Spark ML Events

2018-12-08 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/23263 **[Test build #99868 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99868/testReport)** for PR 23263 at commit