[HIVE-18778] Needs to capture input/output entities in explain

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.3.4, 3.1.1, 3.2.0, 4.0.0-alpha-1
• Target Version/s: 3.2.0, 4.0.0
• Hadoop Flags: Reviewed
• Assignee: Daniel Dai
• Reporter: Daniel Dai
• Created: 23/Feb/18 00:27
• Updated: 21/Oct/22 07:00
• Resolved: 25/Oct/18 18:09

Description

With Sentry enabled, commands like explain drop table foo fail with explain drop table foo;

Error: Error while compiling statement: FAILED: SemanticException No valid privileges
 Required privilege( Table) not available in input privileges
 The required privileges: (state=42000,code=40000)

Sentry fails to authorize because the ExplainSemanticAnalyzer uses an instance of DDLSemanticAnalyzer to analyze the explain query.

BaseSemanticAnalyzer sem = SemanticAnalyzerFactory.get(conf, input);
sem.analyze(input, ctx);
sem.validate()

The inputs/outputs entities for this query are set in the above code. However, these are never set on the instance of ExplainSemanticAnalyzer itself and thus is not propagated into the HookContext in the calling Driver code.

sem.analyze(tree, ctx); --> this results in calling the above code that uses DDLSA
hookCtx.update(sem); --> sem is an instance of ExplainSemanticAnalyzer, this code attempts to update the HookContext with the input/output info from ESA which is never set.

Issue Links

• is duplicated by: HIVE-18829 Inputs/Outputs are not propagated to SA hooks for explain commands.