Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/13794
cc @srowen Can you help close this ? We won't need this feature for now.
---
-
To unsubscribe, e-mail:
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/13794
+1 @jkbradley For now it is better to keep the current implementation for
the 4 meta-algo in pyspark.
@yinxusen Would you mind to close this PR ? But I still appreciate your
contribution
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/13794
@yinxusen Thanks for this PR! I still think this seems like a very cool
feature, but I've become less convinced that it's worth the engineering and
maintenance effort. The alternative to this
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/13794
@yinxusen Hi, are you still working on 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
Github user yinxusen commented on the issue:
https://github.com/apache/spark/pull/13794
Thanks @holdenk Yes, I am still interested in this. @jkbradley Do we still
need the PR to support meta-algorithms in PySpark?
---
If your project is set up for it, you can reply to this email and
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/13794
@yinxusen - is this something you are still interested in? If so updating
it to master would be good as well as making sure the unit tests pass in
jenkins (a lot of reviwers just skip PRs which are
Github user yinxusen commented on the issue:
https://github.com/apache/spark/pull/13794
@jkbradley Update: Now I've added the PythonEstimator and PythonModel. For
PythonEvaluator, it's better to commit in along with changes of CrossValidator.
It's ready to review.
---
If your
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13794
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61197/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13794
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13794
**[Test build #61197 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61197/consoleFull)**
for PR 13794 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13794
**[Test build #61197 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61197/consoleFull)**
for PR 13794 at commit
11 matches
Mail list logo