[
https://issues.apache.org/jira/browse/HIVE-2629?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13181185#comment-13181185
]
Phabricator commented on HIVE-2629:
-----------------------------------
amareshwarisr has commented on the revision "HIVE-2629 [jira] Make a single
Hive binary work with both 0.20.x and 0.23.0".
The directory commonSecure should be changed to common/secure. Why don't we
put those files in the directory common itself? Why create a new directory?
Putting them in common would make the code cleaner
INLINE COMMENTS
build-common.xml:118-123 Why are these changes required? If not required, can
you remove them?
build.properties:13-17 Are we going to add new version here for all the
upcoming versions as well? I don't think we should do it this way.
shims/build.xml:57 Can we change commonSecure to common.secure in all the
places?
REVISION DETAIL
https://reviews.facebook.net/D711
> Make a single Hive binary work with both 0.20.x and 0.23.0
> ----------------------------------------------------------
>
> Key: HIVE-2629
> URL: https://issues.apache.org/jira/browse/HIVE-2629
> Project: Hive
> Issue Type: Bug
> Components: Shims
> Reporter: Carl Steinbach
> Assignee: Thomas Weise
> Fix For: 0.9.0
>
> Attachments: HIVE-2629.D711.1.patch, HIVE-2629.D711.2.patch,
> HIVE-2629.patch
>
>
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira