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
rtpengine: add CRC32 hash algo #2558
Conversation
Thanks! Looks not intrusive, maybe @rfuchs wants to check it, otherwise can be merged from my point of view. |
(cherry picked from commit 1034908)
(cherry picked from commit 1034908)
Thanks for the merge. I've backported it to 5.3 and 5.4 branches. |
@smititelu its a new functionality, and should therefore not backported by default to stable branches, unless it qualifies as a bugfix. |
Like, have the existing algorithm certain problems that are fixed with the new hash algorithm. |
Hi @henningw, We are obtaining smoother rtpengine session distribution results using crc32. Also crc32 is an alternative for those who don't want to load crypto.so module. So, IMHO, this PR fixes the above two. Thank you, |
Hello @smititelu, sorry this does not sound like a bug fix to me. With the same argumentation we could e.g. merge the new dispatcher algorithm 13 to stable, as it provides a better call distribution in certain scenarios. :-) |
Pre-Submission Checklist
in
doc/
subfolder, the README file is autogenerated)Type Of Change
Checklist:
Description
Add CRC32 selection algo.