Github user 10110346 commented on the issue:
https://github.com/apache/spark/pull/23216
Ok, I will close this PR, thank you very much
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/23216
I think just leave it. The `@transient` in `ShuffleMapTasks`'s `locs` is
just superfluous here, not sure it's worth changing.
---
Github user 10110346 commented on the issue:
https://github.com/apache/spark/pull/23216
>
>
> Are you sure it's even a field in the class? it looks like it's only used
to define this:
>
> ```
> @transient private[this] val preferredLocs: Seq[TaskLocation] = {
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/23216
Are you sure it's even a field in the class? it looks like it's only used
to define this:
```
@transient private[this] val preferredLocs: Seq[TaskLocation] = {
if (locs ==
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23216
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/23216
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99674/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23216
**[Test build #99674 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99674/testReport)**
for PR 23216 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23216
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/23216
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/23216
**[Test build #99674 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99674/testReport)**
for PR 23216 at commit
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/23216
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23216
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23216
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99660/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23216
**[Test build #99660 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99660/testReport)**
for PR 23216 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23216
**[Test build #99660 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99660/testReport)**
for PR 23216 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23216
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/23216
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
17 matches
Mail list logo