Remove policy class caching (!15) · Merge requests · GitLab.org / ruby / gems / declarative-policy

Alex Kalderimis requested to merge ajk-remove-caching into main May 20, 2021

This removes policy class caching.

This caching was a defensive measure to mitigate against a ruby bug (https://bugs.ruby-lang.org/issues/11119). This was fixed two years ago, and tests confirm that the test suite is no slower with it removed (0.045 sec now vs 0.050 sec before).

See: https://gitlab.com/gitlab-org/declarative-policy/-/issues/4

Edited May 21, 2021 by Alex Kalderimis

Remove policy class caching

Merge request reports