anujphadke has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/7332

Change subject: IMPALA-5582: Store sentry privileges in lower case
......................................................................

IMPALA-5582: Store sentry privileges in lower case

Privileges granted to a db whose name contains upper case characters can 
disappear
after a few seconds. A privilege is inserted into the catalogObjectCache using 
a key
that uses the db name. The key gets converted to lower case before inserting.
The sentryProxy thread on the other hand returns the db name in lower case. 
When the
catalogObjectCache gets updated and the old catalog object is removed from the 
cache
it ends up deleting the new object since they both use the same key. This 
change stores
the privileges in lower case  which does not trigger these chain on events on a
sentryProxy thread update.

Change-Id: Ide3dfa601fcf77f5acc6adce9bea443aea600901
---
M fe/src/main/java/org/apache/impala/analysis/PrivilegeSpec.java
1 file changed, 3 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/32/7332/1
-- 
To view, visit http://gerrit.cloudera.org:8080/7332
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ide3dfa601fcf77f5acc6adce9bea443aea600901
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: anujphadke <[email protected]>

Reply via email to