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)