[SENTRY-2280] The request received in SentryPolicyStoreProcessor.sentry_notify_hms_event is null

• Type: Sub-task
• Status: Resolved
• Priority: Critical
• Resolution: Fixed
• Affects Version/s: 2.1.0
• Fix Version/s: 2.1.0
• Component/s: Sentry
• Assignee: Na Li
• Reporter: Na Li
• Created: 26/Jun/18 05:08
• Updated: 28/Jun/18 20:27
• Resolved: 28/Jun/18 20:27

Description

When running e2e test TestDbPrivilegeCleanupOnDrop.testRenameTablesWithinDBSinglePrivilege, I found the request received in SentryPolicyStoreProcessor.sentry_notify_hms_event is null

There are multiple issues in fix from "SENTRY-2243: Extend the thrift definition for policy service to learn owner information"

1. The exception was thrown because the protocol_version was not set
2. TSentryAuthorizable.server was not set
3. TSentryHmsEventNotification.ownerType and ownerName are not set for table rename event

As a result, the request received by server is null since thrift at server side cannot re-construct a valid request of type TSentryHmsEventNotification

Once we fix the above issues, we need to make sure SentryPolicyStoreProcessor.sentry_notify_hms_event does not transfer owner in "alter table rename" event.

Issue Links

• links to: Review Link for this case