[GitHub] spark issue #20970: [SPARK-23562][ML] Forward RFormula handleInvalid Param t...

2018-04-03 Thread yogeshg
Github user yogeshg commented on the issue: https://github.com/apache/spark/pull/20970 - [ ] send all PRs against subtasks, rather than against the parent task - [ ] avoid using infix notation for testing `contains` ---

[GitHub] spark issue #20970: [SPARK-23562][ML] Forward RFormula handleInvalid Param t...

2018-04-03 Thread jkbradley
Github user jkbradley commented on the issue: https://github.com/apache/spark/pull/20970 Side note: In general, when there is a JIRA with subtasks, it's nice to send all PRs against subtasks, rather than against the parent task. This can be good when we think of future subtasks which

[GitHub] spark issue #20970: [SPARK-23562][ML] Forward RFormula handleInvalid Param t...

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

[GitHub] spark issue #20970: [SPARK-23562][ML] Forward RFormula handleInvalid Param t...

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

[GitHub] spark issue #20970: [SPARK-23562][ML] Forward RFormula handleInvalid Param t...

2018-04-03 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20970 **[Test build #88854 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88854/testReport)** for PR 20970 at commit [`b53cbfa`](https://github.com/apache/spark/commit/b

[GitHub] spark issue #20970: [SPARK-23562][ML] Forward RFormula handleInvalid Param t...

2018-04-03 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20970 **[Test build #88854 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88854/testReport)** for PR 20970 at commit [`b53cbfa`](https://github.com/apache/spark/commit/b5

[GitHub] spark issue #20970: [SPARK-23562][ML] Forward RFormula handleInvalid Param t...

2018-04-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20970 Can one of the admins verify this patch? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional