Skip to content

Conversation

binarin
Copy link
Contributor

@binarin binarin commented Jan 14, 2016

It prevents atom table overflow in a long running broker.

Fixes #549

It prevents atom table overflow in a long running broker.

Fixes rabbitmq#549
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rabbit_misc:random is undefined.
random:uniform can be used in this case, because it just retries in same process.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, you'll need rabbitmq/rabbitmq-common#48
And also you can't simply run ./scripts/rabbitmqctl from rabbitmq-server checkout - rabbit_common will be missing from code path (but on installed server it will be available through rabbit_common.ez archive).

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One can run make package-generic-unix UNOFFICIAL_RELEASE=true from the umbrella to produce a tarball. It should take a couple of minutes.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Then every sequence of "random" numbers will be the same - and it's exactly the thing that we want to avoid, to reduce contention.

michaelklishin added a commit that referenced this pull request Jan 14, 2016
Limit number of unique node names for rabbitmqctl
@michaelklishin michaelklishin merged commit d54c06a into rabbitmq:stable Jan 14, 2016
lukebakken added a commit to rabbitmq/rabbitmq-cli that referenced this pull request Oct 20, 2020
This will limit the number of CLI node names to 1024 by taking the
current epoch seconds modulo a random number between 1 and 1024.

Follow-up to the following:

rabbitmq/rabbitmq-server#552

https://groups.google.com/d/msg/rabbitmq-users/aSnlC5hSvMk/sIWtv5fxCgAJ
HoloRin pushed a commit to rabbitmq/rabbitmq-monorepo that referenced this pull request Oct 22, 2020
This will limit the number of CLI node names to 1024 by taking the
current epoch seconds modulo a random number between 1 and 1024.

Follow-up to the following:

rabbitmq/rabbitmq-server#552

https://groups.google.com/d/msg/rabbitmq-users/aSnlC5hSvMk/sIWtv5fxCgAJ
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.

3 participants