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

Change directory name generation function for queue indexes and vhosts #1250

Merged
merged 6 commits into from
Jul 4, 2017

Conversation

hairyhum
Copy link
Contributor

@hairyhum hairyhum commented Jun 7, 2017

Partial fix for #1243 for 3.7

It was a mistake to relate on md5(term_to_binary(..)) to generate
vhost and queue directory name, since term_to_binary format can
change.
Migration functions take care of renaming directories.

@hairyhum hairyhum added this to the 3.7.0 milestone Jun 7, 2017
@hairyhum hairyhum force-pushed the rabbitmq-server-1246-master branch from 0ed7568 to 0739a46 Compare June 12, 2017 13:09
Daniil Fedotov added 2 commits June 13, 2017 12:08
It was a mistake to relate on md5(term_to_binary(..)) to generate
vhost and queue directory name, since term_to_binary format can
change.
Migration functions take care of renaming directories.
@hairyhum hairyhum force-pushed the rabbitmq-server-1246-master branch 2 times, most recently from 15dc086 to dd18cf3 Compare June 13, 2017 12:48
@hairyhum hairyhum changed the title DO NOT MERGE Change directory name generation function for queue indexes and vhosts Change directory name generation function for queue indexes and vhosts Jun 15, 2017
@lukebakken lukebakken self-requested a review June 21, 2017 13:15
@michaelklishin michaelklishin merged commit b7f3904 into master Jul 4, 2017
@lukebakken lukebakken deleted the rabbitmq-server-1246-master branch July 24, 2017 18:12
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