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

["Gunther Hagleitner (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/HIVE-9971?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Gunther Hagleitner updated HIVE-9971:
-------------------------------------
    Attachment: HIVE-9971.1.patch

> 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
>         Attachments: HIVE-9971.1.patch
>
>
> 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)