[ https://issues.apache.org/jira/browse/HIVE-405?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12730673#action_12730673 ]
Zheng Shao commented on HIVE-405: --------------------------------- I think Operator.inputObjectInspectors and Operator.parentsObjectInspector contain the same information. So we don't need to add inputObjectInspectors to Operator. > Operators should pass ObjectInspector in init instead of forward > ---------------------------------------------------------------- > > Key: HIVE-405 > URL: https://issues.apache.org/jira/browse/HIVE-405 > Project: Hadoop Hive > Issue Type: Bug > Components: Query Processor > Affects Versions: 0.4.0 > Reporter: Zheng Shao > Assignee: Prasad Chakka > Priority: Critical > Attachments: hive-405.patch > > > We are always passing the same ObjectInspector, so there is no need to pass > it again and again in forward. > Also there is a problem that can ONLY be fixed by passing ObjectInspector in > init: Outer Joins - Outer Joins may not be able to get ObjectInspectors for > all inputs, as a result, there is no way to construct an output > ObjectInspector based on the inputs. Currently we have hard-coded code that > assumes joins are always outputting Strings, which did break but was hidden > by the old framework (because we do toString() when serializing the output, > and toString() is defined for all Java Classes). -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.