[SENTRY-1345] ACLS on table folder disappear after insert for unpartitioned tables

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.7.0
• Fix Version/s: 1.8.0
• Assignee: Sravya Tirukkovalur
• Reporter: Sravya Tirukkovalur
• Created: 20/Jun/16 23:36
• Updated: 29/Jul/16 21:24
• Resolved: 21/Jun/16 20:29

Description

This seems to be due to the fact that onAlterTableEvent is being trigged for this operation of "INSERT on a unpartitioned table" in Hive. And sentry- hive plugin thinks this is an alter table location command, and we add the new path and delete the old path for this Hive object. As add happens first and then the delete on the same path in this case, the path is lost resulting in no acls on this path.
Workaround: This gets corrected at the next HMS restart.

We can fix on Sentry side to not do add/delete if oldPath == newPath. But we also need to understand the Hive behavior.
Not exactly sure when this regressed as we dont seem to have coverage for it. Also adding a test case as part of the patch which can be used against older versions.

Issue Links

• is related to: SENTRY-1376 Fix alter property case correctly - Deletes ACLS on the table