This commit fixes two issues:

1. ||= with a class instance variable is not thread-safe. We now
just avoid the memoization and use constants.
2. get_store_class was being called with symbols in some cases and
strings in other cases. We now standardize using symbols.

Relates to https://gitlab.com/gitlab-org/gitlab/-/issues/271253