Uploaded image for project: 'Hive'
  1. Hive
  2. HIVE-11193

ConstantPropagateProcCtx should use a Set instead of a List to hold operators to be deleted

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.0.0
    • Component/s: Logical Optimizer
    • Labels:
      None

      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.

        Attachments

        1. HIVE-11193.03.patch
          9 kB
          Wei Zheng
        2. HIVE-11193.02.patch
          9 kB
          Wei Zheng
        3. HIVE-11193.01.patch
          2 kB
          Wei Zheng

          Activity

            People

            • Assignee:
              wzheng Wei Zheng
              Reporter:
              wzheng Wei Zheng
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: