[GitHub] spark issue #12217: [WIP][SPARK-14408][CORE] Changed RDD.treeAggregate to us...

Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/12217 Uh wait. It actually pass the tests after updating this with the current master ... and even I fixed this before -

[GitHub] spark issue #12217: [WIP][SPARK-14408][CORE] Changed RDD.treeAggregate to us...

Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/12217 The tests in `AFTSurvivalRegressionSuite` was being failed. `treeAggregate` was being called for AFT cost. ``` - aft survival regression: default params 0.0 equaled 0.0

[GitHub] spark issue #12217: [WIP][SPARK-14408][CORE] Changed RDD.treeAggregate to us...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/12217 **[Test build #3773 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3773/testReport)** for PR 12217 at commit

[GitHub] spark issue #12217: [WIP][SPARK-14408][CORE] Changed RDD.treeAggregate to us...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/12217 **[Test build #3773 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3773/testReport)** for PR 12217 at commit

[GitHub] spark issue #12217: [WIP][SPARK-14408][CORE] Changed RDD.treeAggregate to us...

Github user NathanHowell commented on the issue: https://github.com/apache/spark/pull/12217 Nothing looks obviously broken, their combiner looks fine. Rerunning the tests would help. On Jun 2, 2017 07:02, "Hyukjin Kwon" wrote: > Hi

[GitHub] spark issue #12217: [WIP][SPARK-14408][CORE] Changed RDD.treeAggregate to us...

Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/12217 Hi @jkbradley and @srowen, could we retest this just to see the error messages? It looks the last test results are not accessible (to me). --- If your project is set up for it, you can reply

[GitHub] spark issue #12217: [WIP][SPARK-14408][CORE] Changed RDD.treeAggregate to us...

Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/12217 @NathanHowell, do you maybe have any idea on this (sorry, probably wrong person to cc but I know no one I could think ... )? --- If your project is set up for it, you can reply to this email

[GitHub] spark issue #12217: [WIP][SPARK-14408][CORE] Changed RDD.treeAggregate to us...

Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/12217 I remember I took a look for this (in the last time while looking at stale PRs) and I remember I had no idea as well ... @jkbradley I was just wondering if we should better leave this closed