You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by "Gunther Hagleitner (JIRA)" <ji...@apache.org> on 2015/03/16 06:36:38 UTC

[jira] [Created] (HIVE-9971) Clean up operator class

Gunther Hagleitner created HIVE-9971:
----------------------------------------

             Summary: Clean up operator class
                 Key: HIVE-9971
                 URL: https://issues.apache.org/jira/browse/HIVE-9971
             Project: Hive
          Issue Type: Bug
            Reporter: Gunther Hagleitner
            Assignee: Gunther Hagleitner


This is mostly cleanup although it does enhance the pipeline in one respect. It introduces asyn init for operators and uses it for hash table loading where desired.

There's a bunch of weird code associated with the operator class:

- initialize isn't recursive, rather initializeOp is supposed to call initializeChildren. That has led to bugs in the past.

- setExecContext and passExecContext. Both are recursive, but passExecContext calls setExecContext and then recurses again. Boo.

- lots of (getChildren() != null) although that can't happen anymore

- TezCacheAccess is a hack. We should just leave init of inputs up to the operator that needs it.

- Need some sanity checks that make sure that operators were all initialized.





--
This message was sent by Atlassian JIRA
(v6.3.4#6332)