Github user ConeyLiu commented on the issue:
https://github.com/apache/spark/pull/19661
thanks everyone.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19661
thanks, merging to master!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19661
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19661
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83676/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19661
**[Test build #83676 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83676/testReport)**
for PR 19661 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19661
**[Test build #83676 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83676/testReport)**
for PR 19661 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19661
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19661
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83665/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19661
**[Test build #83665 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83665/testReport)**
for PR 19661 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19661
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19661
**[Test build #83665 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83665/testReport)**
for PR 19661 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19661
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19661
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83642/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19661
**[Test build #83642 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83642/testReport)**
for PR 19661 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19661
**[Test build #83642 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83642/testReport)**
for PR 19661 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19661
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user ConeyLiu commented on the issue:
https://github.com/apache/spark/pull/19661
Thanks for reviewing. The code is updated.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user ConeyLiu commented on the issue:
https://github.com/apache/spark/pull/19661
>So why don't you include some classes such as
org.apache.spark.ml.feature.Instance ?
I'm not family with those algorithm, I can add them such as
`org.apache.spark.ml.feature.Instance` .
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/19661
And I don't know whether these class dependency injection into spark-core
lib is reasonable ...
---
-
To unsubscribe,
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/19661
So why do you include the class such as
`org.apache.spark.ml.feature.Instance`.
You can look into a lot of algos, in `ml` package (not `mllib`), still use
something like `RDD[Instance]`
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19661
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user ConeyLiu commented on the issue:
https://github.com/apache/spark/pull/19661
#19586
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
22 matches
Mail list logo