URL: https://github.com/SSSD/sssd/pull/1018 Title: #1018: mem-cache: sizes of free and data tables were made consistent
alexey-tikhonov commented: """
but I have to admit that I would prefer to have a more minimal patch for 1-16 which only fixes the too small free table issue. E.g. something like `mc_ctx->ft_size = MC_FT_SIZE(mc_ctx->dt_size);` (not tested).
`ft_size` should be based on amount of slots (not bytes) it data table.
But most importantly, could you please explain what is the functional difference / what sense does it make to stick to misleading `SSS_AVG_*_PAYLOAD`?
I think there is (almost) no change from user perspective / observed behavior. But code is simpler/cleaner and thus easier to maintain.
I also think it makes sense for "core" components to keep difference between 1-16 and master branch small (where possible). """
See the full comment at https://github.com/SSSD/sssd/pull/1018#issuecomment-610228026