[ 
https://issues.apache.org/jira/browse/HIVE-4810?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13704035#comment-13704035
 ] 

Edward Capriolo commented on HIVE-4810:
---------------------------------------

I do not think we are going about this the correct way. This is an excessively 
large patch on one of the keys pieces of hive. I dont think this type of thing 
should be done in 10 days with the +1 of only one committer, for the sole 
purpose of making it easier to add new cool feature x , while simultaniously 
sending a big fu to all the other people that have patches waiting for review 
for months that they will now have to rebase.

I am not sure anyone has sold tez to the committers in a way that we should 
refactor key components of hive just so we can play with it.
                
> Refactor exec package
> ---------------------
>
>                 Key: HIVE-4810
>                 URL: https://issues.apache.org/jira/browse/HIVE-4810
>             Project: Hive
>          Issue Type: Bug
>            Reporter: Gunther Hagleitner
>            Assignee: Gunther Hagleitner
>             Fix For: 0.12.0
>
>         Attachments: HIVE-4810.1.patch, HIVE-4810.2.patch, HIVE-4810.3.patch, 
> HIVE-4810.4.patch, HIVE-4810.5.patch, HIVE-4810.6.patch, HIVE-4810.7.patch
>
>
> The exec package contains both operators and classes used to execute the job. 
> Moving the latter into a sub package makes the package slightly more 
> manageable and will make it easier to provide a tez-based implementation. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to