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: reviews-h..
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 comma
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
[`77f994f`](https://github.com/apache/spark/commit/7
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
[`77f994f`](https://github.com/apache/spark/commit/77
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 comma
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
[`d7090bb`](https://github.com/apache/spark/commit/d
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: reviews-h...@spark.apa
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
[`d7090bb`](https://github.com/apache/spark/commit/d7
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 comma
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
[`2eb1b62`](https://github.com/apache/spark/commit/2
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
[`2eb1b62`](https://github.com/apache/spark/commit/2e
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: reviews-h...@spa
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 com
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, e-mai
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]` and
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: reviews-h...@spark.a
22 matches
Mail list logo