Skip to content

md5Hex no longer works in MVEL scripts (taken out 2014-07-29) #5803

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

Closed
AaronM04 opened this issue Apr 14, 2014 · 4 comments
Closed

md5Hex no longer works in MVEL scripts (taken out 2014-07-29) #5803

AaronM04 opened this issue Apr 14, 2014 · 4 comments

Comments

@AaronM04
Copy link

I apologize if an issue already exists for this; I am having trouble searching for issues...

I noticed that this code was removed in a "cleanup" commit ( 86ceabf by @kimchy ), apparently under the impression that it was only used for tests. People have been using it in their production MVEL scripts for pseudo-random shuffling. Is this something we should bring back? Is there another way to do pseudo-random shuffling (I tried using function_score, but could not get it to work)?

@spinscale
Copy link
Contributor

maybe you can show an example how you use it for pseudo-random shuffling and we can check, if you can have the same behaviour with function_score, before bringing it back (seems cleaner with function score from my outside point of view)?

@imotov
Copy link
Contributor

imotov commented Apr 17, 2014

I think figuring why function score doesn't work is the way to go. However, you can also simply implement the functionality of the old md5Hex function in your script.

@AaronM04
Copy link
Author

I withdraw my request. :-)

@clintongormley
Copy link
Contributor

Not an issue. closing

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

No branches or pull requests

4 participants