You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@celeborn.apache.org by "turboFei (via GitHub)" <gi...@apache.org> on 2025/02/24 23:49:17 UTC
[PR] [CELEBORN-1878] Shuffle the available workers to randomize the order for rack awareness during slot offering [celeborn]
turboFei opened a new pull request, #3116:
URL: https://github.com/apache/celeborn/pull/3116
<!--
Thanks for sending a pull request! Here are some tips for you:
- Make sure the PR title start w/ a JIRA ticket, e.g. '[CELEBORN-XXXX] Your PR title ...'.
- Be sure to keep the PR description updated to reflect all changes.
- Please write your PR title to summarize what this PR proposes.
- If possible, provide a concise example to reproduce the issue for a faster review.
-->
### What changes were proposed in this pull request?
### Why are the changes needed?
### Does this PR introduce _any_ user-facing change?
### How was this patch tested?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@celeborn.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CELEBORN-1878] Support to shuffle the available workers to randomize the order for rack awareness during slot assign [celeborn]
Posted by "turboFei (via GitHub)" <gi...@apache.org>.
turboFei commented on PR #3116:
URL: https://github.com/apache/celeborn/pull/3116#issuecomment-2680314550
Will try to investigate the iteration Order of hashSet. the `shuffle` might be not needed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@celeborn.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CELEBORN-1878] Support to shuffle the available workers to randomize the order for rack awareness during slot assign [celeborn]
Posted by "turboFei (via GitHub)" <gi...@apache.org>.
turboFei closed pull request #3116: [CELEBORN-1878] Support to shuffle the available workers to randomize the order for rack awareness during slot assign
URL: https://github.com/apache/celeborn/pull/3116
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@celeborn.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CELEBORN-1878] Shuffle the available workers to randomize the order for rack awareness during slot offering [celeborn]
Posted by "turboFei (via GitHub)" <gi...@apache.org>.
turboFei commented on PR #3116:
URL: https://github.com/apache/celeborn/pull/3116#issuecomment-2680052019
Seems the test results not bad without shuffling the available workers:
```
Shuffle workers: true, selected workers per AZ
[{az0=69, az1=22, az2=54, az3=260, az4=290, az5=105}, {az0=68, az1=27, az2=56, az3=268, az4=283, az5=98}, {az0=80, az1=32, az2=56, az3=226, az4=276, az5=130}, {az0=66, az1=22, az2=48, az3=244, az4=302, az5=118}, {az0=67, az1=23, az2=51, az3=240, az4=313, az5=106}]
Shuffle workers: false, selected workers per AZ
[{az0=30, az2=100, az3=211, az4=273, az5=186}, {az0=100, az2=100, az3=199, az4=301, az5=100}, {az0=108, az2=86, az3=253, az4=228, az5=125}, {az0=110, az2=86, az3=242, az4=230, az5=132}, {az2=152, az3=231, az4=317, az5=100}]
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@celeborn.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CELEBORN-1878] Support to shuffle the available workers to randomize the order for rack awareness during slot assign [celeborn]
Posted by "turboFei (via GitHub)" <gi...@apache.org>.
turboFei commented on PR #3116:
URL: https://github.com/apache/celeborn/pull/3116#issuecomment-2680237184
cc @AngersZhuuuu @RexXiong @pan3793
How do you think about?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@celeborn.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org