[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-22 Thread srowen
Github user srowen commented on the issue:

https://github.com/apache/spark/pull/22875
  
Merged to master


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99174/
Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Merged build finished. Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-22 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/22875
  
**[Test build #99174 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99174/testReport)**
 for PR 22875 at commit 
[`692ff63`](https://github.com/apache/spark/commit/692ff63073441fb74975f281798ecb8d615ff229).
 * This patch passes all tests.
 * This patch merges cleanly.
 * This patch adds the following public classes _(experimental)_:
  * `  class RuleSummary(`
  * `class QueryPlanningTracker `
  * `class QueryExecution(`


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 

https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5274/
Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-22 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/22875
  
**[Test build #99174 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99174/testReport)**
 for PR 22875 at commit 
[`692ff63`](https://github.com/apache/spark/commit/692ff63073441fb74975f281798ecb8d615ff229).


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Merged build finished. Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-22 Thread mgaido91
Github user mgaido91 commented on the issue:

https://github.com/apache/spark/pull/22875
  
sure, thanks @srowen , no need to apologize at all, thanks for your help 
reviewing this.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99133/
Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Merged build finished. Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-21 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/22875
  
**[Test build #99133 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99133/testReport)**
 for PR 22875 at commit 
[`2b18e55`](https://github.com/apache/spark/commit/2b18e555d34b7fd2a4fc8048a7d7f249161117bb).
 * This patch passes all tests.
 * This patch merges cleanly.
 * This patch adds no public classes.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Merged build finished. Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 

https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5240/
Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-21 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/22875
  
**[Test build #99133 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99133/testReport)**
 for PR 22875 at commit 
[`2b18e55`](https://github.com/apache/spark/commit/2b18e555d34b7fd2a4fc8048a7d7f249161117bb).


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-21 Thread mgaido91
Github user mgaido91 commented on the issue:

https://github.com/apache/spark/pull/22875
  
retest this please


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Test FAILed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99123/
Test FAILed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Merged build finished. Test FAILed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-21 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/22875
  
**[Test build #99123 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99123/testReport)**
 for PR 22875 at commit 
[`2b18e55`](https://github.com/apache/spark/commit/2b18e555d34b7fd2a4fc8048a7d7f249161117bb).
 * This patch **fails Spark unit tests**.
 * This patch merges cleanly.
 * This patch adds no public classes.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-21 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/22875
  
**[Test build #99123 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99123/testReport)**
 for PR 22875 at commit 
[`2b18e55`](https://github.com/apache/spark/commit/2b18e555d34b7fd2a4fc8048a7d7f249161117bb).


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Merged build finished. Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 

https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5229/
Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-21 Thread mgaido91
Github user mgaido91 commented on the issue:

https://github.com/apache/spark/pull/22875
  
retest this please


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Test FAILed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99109/
Test FAILed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Merged build finished. Test FAILed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-21 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/22875
  
**[Test build #99109 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99109/testReport)**
 for PR 22875 at commit 
[`2b18e55`](https://github.com/apache/spark/commit/2b18e555d34b7fd2a4fc8048a7d7f249161117bb).
 * This patch **fails Spark unit tests**.
 * This patch merges cleanly.
 * This patch adds no public classes.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 

https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5221/
Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Merged build finished. Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-21 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/22875
  
**[Test build #99109 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99109/testReport)**
 for PR 22875 at commit 
[`2b18e55`](https://github.com/apache/spark/commit/2b18e555d34b7fd2a4fc8048a7d7f249161117bb).


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-21 Thread mgaido91
Github user mgaido91 commented on the issue:

https://github.com/apache/spark/pull/22875
  
thanks for your review @srowen 


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Merged build finished. Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99057/
Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-20 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/22875
  
**[Test build #99057 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99057/testReport)**
 for PR 22875 at commit 
[`f09e985`](https://github.com/apache/spark/commit/f09e985d837155e26dde51381c400a10a03a0fa9).
 * This patch passes all tests.
 * This patch merges cleanly.
 * This patch adds no public classes.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-20 Thread srowen
Github user srowen commented on the issue:

https://github.com/apache/spark/pull/22875
  
Ah OK I wasn't reading that carefully.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-20 Thread mgaido91
Github user mgaido91 commented on the issue:

https://github.com/apache/spark/pull/22875
  
Yes, it was deprecated in #22756 and it is deprecated since 3.0, so we 
cannot remove it...


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-20 Thread srowen
Github user srowen commented on the issue:

https://github.com/apache/spark/pull/22875
  
How about BisecingKMeansModel? That's what I was looking at. That 
computeCost is also deprecated.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-20 Thread mgaido91
Github user mgaido91 commented on the issue:

https://github.com/apache/spark/pull/22875
  
Yes, the problem is that there is also the `computeCost` of 
`BisectingKMeans`. I proposed to deprecate it in 2.4 and remove in 3.0, but I 
didn't manage to have it done for 2.4 (please refer to the discussions on 
https://github.com/apache/spark/pull/22764 and 
https://github.com/apache/spark/pull/22756 for the details). So `computeCost` 
of `BisectingKMeans` cannot be removed in 3.0 (unfortunately). The examples for 
KMeans have already been updated using ClusteringEvaluator in #19676.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-20 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/22875
  
**[Test build #99057 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99057/testReport)**
 for PR 22875 at commit 
[`f09e985`](https://github.com/apache/spark/commit/f09e985d837155e26dde51381c400a10a03a0fa9).


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 

https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5180/
Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Merged build finished. Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-20 Thread mgaido91
Github user mgaido91 commented on the issue:

https://github.com/apache/spark/pull/22875
  
retest this please


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Test FAILed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99048/
Test FAILed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Merged build finished. Test FAILed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-20 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/22875
  
**[Test build #99048 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99048/testReport)**
 for PR 22875 at commit 
[`f09e985`](https://github.com/apache/spark/commit/f09e985d837155e26dde51381c400a10a03a0fa9).
 * This patch **fails Spark unit tests**.
 * This patch merges cleanly.
 * This patch adds no public classes.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Merged build finished. Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 

https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5176/
Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-20 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/22875
  
**[Test build #99048 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99048/testReport)**
 for PR 22875 at commit 
[`f09e985`](https://github.com/apache/spark/commit/f09e985d837155e26dde51381c400a10a03a0fa9).


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-20 Thread mgaido91
Github user mgaido91 commented on the issue:

https://github.com/apache/spark/pull/22875
  
any commetn on this @srowen @holdenk ?


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98874/
Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Merged build finished. Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-15 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/22875
  
**[Test build #98874 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98874/testReport)**
 for PR 22875 at commit 
[`6e2d501`](https://github.com/apache/spark/commit/6e2d5018bb0697badc99683ebfbbb49ac13056ec).
 * This patch passes all tests.
 * This patch merges cleanly.
 * This patch adds no public classes.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Merged build finished. Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 

https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5054/
Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-15 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/22875
  
**[Test build #98874 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98874/testReport)**
 for PR 22875 at commit 
[`6e2d501`](https://github.com/apache/spark/commit/6e2d5018bb0697badc99683ebfbbb49ac13056ec).


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98680/
Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Merged build finished. Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-10 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/22875
  
**[Test build #98680 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98680/testReport)**
 for PR 22875 at commit 
[`c2fdddc`](https://github.com/apache/spark/commit/c2fdddcadae2c0297554742e2fd19441e63c5eb0).
 * This patch passes all tests.
 * This patch merges cleanly.
 * This patch adds no public classes.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Merged build finished. Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-10 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/22875
  
**[Test build #98680 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98680/testReport)**
 for PR 22875 at commit 
[`c2fdddc`](https://github.com/apache/spark/commit/c2fdddcadae2c0297554742e2fd19441e63c5eb0).


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 

https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4912/
Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-11-05 Thread mgaido91
Github user mgaido91 commented on the issue:

https://github.com/apache/spark/pull/22875
  
cc @holdenk @srowen 


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-10-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98212/
Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-10-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Merged build finished. Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-10-29 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/22875
  
**[Test build #98212 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98212/testReport)**
 for PR 22875 at commit 
[`62f8606`](https://github.com/apache/spark/commit/62f8606182b79231340e30c4a8eba7b5f9ed819b).
 * This patch passes all tests.
 * This patch merges cleanly.
 * This patch adds no public classes.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-10-29 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/22875
  
**[Test build #98212 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98212/testReport)**
 for PR 22875 at commit 
[`62f8606`](https://github.com/apache/spark/commit/62f8606182b79231340e30c4a8eba7b5f9ed819b).


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-10-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 

https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4591/
Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-10-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Merged build finished. Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-10-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Merged build finished. Test FAILed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-10-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 

https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4590/
Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-10-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Merged build finished. Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-10-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Test FAILed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98210/
Test FAILed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-10-29 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/22875
  
**[Test build #98210 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98210/testReport)**
 for PR 22875 at commit 
[`9ee93b1`](https://github.com/apache/spark/commit/9ee93b14a838fda3b61d36c39e5ee77c032c8fb8).
 * This patch **fails MiMa tests**.
 * This patch merges cleanly.
 * This patch adds no public classes.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-10-29 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/22875
  
**[Test build #98210 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98210/testReport)**
 for PR 22875 at commit 
[`9ee93b1`](https://github.com/apache/spark/commit/9ee93b14a838fda3b61d36c39e5ee77c032c8fb8).


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-10-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 

https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4586/
Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-10-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Merged build finished. Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-10-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Merged build finished. Test FAILed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-10-29 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/22875
  
**[Test build #98201 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98201/testReport)**
 for PR 22875 at commit 
[`9ee93b1`](https://github.com/apache/spark/commit/9ee93b14a838fda3b61d36c39e5ee77c032c8fb8).
 * This patch **fails build dependency tests**.
 * This patch merges cleanly.
 * This patch adds no public classes.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-10-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/22875
  
Test FAILed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98201/
Test FAILed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-10-29 Thread mgaido91
Github user mgaido91 commented on the issue:

https://github.com/apache/spark/pull/22875
  
retest this please


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #22875: [SPARK-25867][ML] Remove KMeans computeCost

2018-10-29 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/22875
  
**[Test build #98201 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98201/testReport)**
 for PR 22875 at commit 
[`9ee93b1`](https://github.com/apache/spark/commit/9ee93b14a838fda3b61d36c39e5ee77c032c8fb8).


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org