Is it a bug of not checking against max bucket and lock power ?

[wenhaocs]

Is it a bug of not checking against max bucket and lock power ?

[sathyaphoenix]

@wenhaocs good catch. Do you mind sending a PR to address that and I can merge it.

[wenhaocs]

@sathyaphoenix Please let me know if you want to throw error explicitly and tell users the limit for #cacheEntries, or just implicitly set the bucketPower to be maximum like in the PR.

[jiayuebao]

@wenhaocs Shall we use std::min instead of std::max to pick the smaller value?

[sathyaphoenix]

To be consistent with the constructor, it makes sense to throw an error when the value is going to be incorrect rather than silently fix it. Most likely in this case the user gave a wrong input and it is better to abort rather than silently run with a misconfigured hash table. Also, like @jiayuebao pointed out, we should be doing a std::min instead of max if we were to adjust to a meaningful value.

@wenhaocs can you add a simple unit test for this ? we can introduce a new .cpp here to add CacheAllocatorConfig unit tests. @jiayuebao please let us know if there is a better place to add the test.

[jiayuebao]

For unit test, we already have one here. And we can add more cases there.