Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21448
**[Test build #92437 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92437/testReport)**
for PR 21448 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21448
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/553/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21448
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/21448
Cool, let me know when you get a chance to update the PR @jaceklaskowski :)
---
-
To unsubscribe, e-mail:
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/21448
@jaceklaskowski how would you feel about just moving abs as suggested by
@rxin?
---
-
To unsubscribe, e-mail:
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/21448
So, I personally don't have strong feelings about which groups these
functions should be in, but just a reminder we should unify with PySpark with
whatever our decision is.
In PySpark
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/21448
I'd only move abs and nothing else.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/21448
If itâs ok Iâll take a quick look at this on Friday as discussed with
Jacek.
---
-
To unsubscribe, e-mail:
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/21448
`abs` does seem to belong with math functions.
`isnan` and `nanvl` are sort of math related but to me feel more like
`isnull`. It's more about testing representations of a value, not math
Github user jaceklaskowski commented on the issue:
https://github.com/apache/spark/pull/21448
It is such a small change that I don't think it's going to take long to get
merged. Reaching out to friendly folks to reach a consensus on it :) /cc
@srowen @holdenk
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21448
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91250/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21448
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21448
**[Test build #91250 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91250/testReport)**
for PR 21448 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21448
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/3663/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21448
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21448
**[Test build #91250 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91250/testReport)**
for PR 21448 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21448
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/21448
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/3660/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21448
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91247/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21448
**[Test build #91247 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91247/testReport)**
for PR 21448 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21448
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21448
**[Test build #91247 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91247/testReport)**
for PR 21448 at commit
22 matches
Mail list logo