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

Gunther Hagleitner commented on HIVE-4810:
------------------------------------------

[~appodictic] have you looked at the changes? I know there's quite a few files 
that are being touched, but the changes are rather simple and straight forward 
in my opinion. The reason so many files have changed is simply because the full 
path names of some classes is recorded in some golden files. No core logic has 
changed. Even if you have to rebase against it that shouldn't be hard. 

As I pointed out in the description, I think grouping some logically cohesive 
units into its own package has value in itself, which is why I pitched it for 
inclusion into the trunk as opposed to the branch. 
 
                
> 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