SolidWallOfCode commented on pull request #7350: URL: https://github.com/apache/trafficserver/pull/7350#issuecomment-732343561
Looks reasonable. IIRC the main reason the ram cache key is 32 bits is because no one was willing to change it when we moved to 64 bit support. I would like, however, to not use "uint64_t" but put in a `using` like `using RamCacheKey = uint64_t;`. I think this will make the code clearer and make it easier to tweak later. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
