moonchen commented on PR #13424: URL: https://github.com/apache/trafficserver/pull/13424#issuecomment-5243639407
Do you only want a cache key that doesn't change depending on the ordering of parameters, or do you also want the origin server to see sorted parameters? If it's only the first one you need, I think an additive hash of parameters will drop the ordering of parameters more efficiently, with fewer allocations. If you also need the sorted parameters to go to the origin, then one suggestion is to check if the parameters are already in sorted order before allocating memory to sort them. In either case, I want to also point out that `cachekey.so` with `--sort-params=true` deduplicates parameters by name, so this behavior is different. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
