Github user squito commented on the issue:
https://github.com/apache/spark/pull/22818
@kiszk I've updated this to cover more cases. I didn't cover some of them
in mllib-local, as ByteArrayMethods isn't visible there, and it would really
only very slightly improve an error msg, so did
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22818
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98274/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22818
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22818
**[Test build #98274 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98274/testReport)**
for PR 22818 at commit
[`3d77303`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22818
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/22818
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/4633/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22818
**[Test build #98274 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98274/testReport)**
for PR 22818 at commit
[`3d77303`](https://github.com/apache/spark/commit/3d
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/22818
Since this PR is not a blocker for 2.4, I think that it would be good to
address these issues as possible.
---
-
To unsubscribe, e
Github user squito commented on the issue:
https://github.com/apache/spark/pull/22818
Actually there are quite a few more uses, even of `Int.MaxValue`, which I
find suspicious, but for the moment I only wanted to touch the cases I
understood better. For example,
["spark.sql.sortMerg
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/22818
Thanks, would it be also possible to double-check `Integer.MAX_VALUE` if
you have not checked yet?
---
-
To unsubscribe, e-mail: r
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22818
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/22818
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97987/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22818
**[Test build #97987 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97987/testReport)**
for PR 22818 at commit
[`64b5ed4`](https://github.com/apache/spark/commit/6
Github user wypoon commented on the issue:
https://github.com/apache/spark/pull/22818
Looks good to me. I reran the test that encountered this issue on a secure
cluster after deploying a build with this change and now it passes.
---
--
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/22818
> JVMs don't you allocate arrays
You grammar there.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Fo
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22818
**[Test build #97987 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97987/testReport)**
for PR 22818 at commit
[`64b5ed4`](https://github.com/apache/spark/commit/64
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22818
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/22818
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/4450/
Github user squito commented on the issue:
https://github.com/apache/spark/pull/22818
@kiszk
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apa
19 matches
Mail list logo