[HIVE-11193] ConstantPropagateProcCtx should use a Set instead of a List to hold operators to be deleted

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.0.0
• Component/s: Logical Optimizer
• Assignee: Wei Zheng
• Reporter: Wei Zheng
• Created: 07/Jul/15 20:32
• Updated: 16/Feb/16 23:50
• Resolved: 09/Jul/15 23:53

Description

During Constant Propagation optimization, sometimes a node ends up being added to opToDelete list more than once.

Later in ConstantPropagate transform, we try to delete that operator multiple times, which will cause SemanticException since the node has already been removed in an earlier pass.

The data structure for storing opToDelete is List. We should use Set to avoid the problem.