Github user srowen commented on the issue:
https://github.com/apache/spark/pull/19126
No, no change has been made. @BryanCutler suggests that the change isn't
needed in `master` but could be applied to `branch-2.2`. To do that you need to
reopen a new PR against that branch. It still
Github user marktab commented on the issue:
https://github.com/apache/spark/pull/19126
@srowen since I am a new to this review process, should I be seeing the
change at http://spark.apache.org/docs/latest/ml-pipeline.html
---
Github user marktab commented on the issue:
https://github.com/apache/spark/pull/19126
Thanks @srowen
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/19126
@marktab you can either close this or reopen vs branch-2.2
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/19126
Yeah, I checked and this is not a problem in master since #17849
---
-
To unsubscribe, e-mail:
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/19126
So I'm wondering if @BryanCutler's change for copying the params means that
this should only be applied to old branches (2.2)?
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19126
**[Test build #81413 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81413/testReport)**
for PR 19126 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19126
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81413/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19126
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19126
**[Test build #81413 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81413/testReport)**
for PR 19126 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19126
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
11 matches
Mail list logo