mirror of
https://github.com/scylladb/scylladb.git
synced 2026-06-02 13:06:57 +00:00
It makes little sense to have the same value for permissions_update_interval_in_ms and permissions_validity_in_ms. This may cause the values to be invalidated only because some minor delays in the timer scheduling. It makes a lot more sense to make the permissions_update_interval_in_ms value smaller than permissions_validity_in_ms. This way we would minimize the chances of "false invalidation" due to some small delays in the timer scheduling. In addition, 2s seems to be a too small value for permissions_validity_in_ms since our default read_request_timeout_in_ms is 5s. This means that a single system_auth read failure would guarantee that the following queries are going to read system_auth data in the foreground. Setting it to 10s would allow a second read attempt before we enforce the foreground read. Signed-off-by: Vlad Zolotarov <vladz@scylladb.com>