Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Provide hashedName in BackplaneStatus Queue Name #1616

Merged
merged 1 commit into from
Jan 22, 2024

Conversation

werkt
Copy link
Collaborator

@werkt werkt commented Jan 22, 2024

Avoid stripping the existingHash, used to calculate the slot with suffix modifiers for queue balancing. Client presentation may also move to presenting each balanced queue name, removing the coordinated calculation on the client.

The name could only have been used by a client prior for name presentation or determination of queue names in redis from slot arrangement.

Avoid stripping the existingHash, used to calculate the slot with suffix
modifiers for queue balancing. Client presentation may also move to
presenting each balanced queue name, removing the coordinated
calculation on the client.

The name could only have been used by a client prior for name
presentation or determination of queue names in redis from slot
arrangement.
@werkt werkt merged commit ea35e56 into bazelbuild:main Jan 22, 2024
2 checks passed
@werkt werkt deleted the status-hashed-name branch January 22, 2024 15:01
amishra-u pushed a commit to amishra-u/bazel-buildfarm that referenced this pull request Feb 1, 2024
Avoid stripping the existingHash, used to calculate the slot with suffix
modifiers for queue balancing. Client presentation may also move to
presenting each balanced queue name, removing the coordinated
calculation on the client.

The name could only have been used by a client prior for name
presentation or determination of queue names in redis from slot
arrangement.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant