Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

fixed log message for privilege revocation #20

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

siw36
Copy link

@siw36 siw36 commented Oct 8, 2019

When revokeAllPrivilegesIfRequired gets called, the variable consecutiveUpdateFailuresCount does get set to 0 before the log message is written.

This causes the logs to always show that the cache update failed for 0 times:

RROR org.apache.sentry.provider.db.generic.UpdatableCache: Failed to update roles and privileges cache for 0 times. Revoking all privileges from cache, which will cause all authorization requests to fail.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants