Adenilson Cavalcanti
Adenilson Cavalcanti
Hello there, Orvid! Thanks for reading the patch, I appreciate it. I don't expect the current patch to land as it is now. The primary objective was to kick start...
Adding some people who have being active on the hashes folder: @yfeldblum, @terrelln Any input on the ideas I shared above? @Orvid: I've just added on this patch an initial...
So the idea of a new interface for a faster hasher would look similar to (draft, incomplete): https://gist.github.com/Adenilson/2111f3a1f33b671798a380b67dd36c9a Meanwhile, I'm moving the changes on the Json unit test (i.e. re-enable...
The small changes on the json utest is on #1812.
Updated patch, changes: a) Keep the deprecated hashes (i.e. Farmhash, SpookyHash, etc). b) Changes on the JSON unit test were split to the other independent merge request #1812. c) Migrate...
Fixed typo in the CL msg.
@Orvid touching the base to ask how the internal testing went and how much breakage the current version of the patch caused (hopefully none...). :-) I can next update the...
@Orvid Please see the updated version using the system's provided xxhash library as suggested. By the way, do you have the results of the previous version after the required internal...
@Orvid did you have time to verify the latest version of the patch (i.e. where we use the xxhash lib available in the system as suggested)?
@aaronabramov, @yfeldblum and @ispeters any feedback? The idea is to re-enable compilation of the main JSON unit test and add two new unit tests that could be used for a...