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
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
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
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
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):
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
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,
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,
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:
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
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
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
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
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
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):
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:
19 matches
Mail list logo