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

Phabricator commented on HBASE-4867:
------------------------------------

todd has commented on the revision "[jira] [HBASE-4867] A tool to merge 
configuration files".

  It seems it would be preferable to write this in Java rather than Python, 
since nothing else in HBase is written in python. But, failing a complete 
rewrite, I'd recommend you drop use of the with statement so it works in 
py2.4/RHEL5. Should be a trivial change to try..finally

INLINE COMMENTS
  src/main/python/hbase/merge_conf.py:8 This isn't compatible with Python 2.4, 
which is the latest version on RHEL5. Most of our customers still run RHEL5, 
rather than RHEL6, so this script wouldn't be compatible.

REVISION DETAIL
  https://reviews.facebook.net/D537

                
> A tool to merge configuration files
> -----------------------------------
>
>                 Key: HBASE-4867
>                 URL: https://issues.apache.org/jira/browse/HBASE-4867
>             Project: HBase
>          Issue Type: New Feature
>            Reporter: Mikhail Bautin
>            Assignee: Mikhail Bautin
>            Priority: Minor
>         Attachments: D537.1.patch
>
>
> With our cluster configuration setup it would be good to have a tool that 
> would merge HBase configuration, so that files appearing later in the list 
> would override properties specified in earlier files. This way we could merge 
> application-specific configuration file with the cluster-specific 
> configuration file (with the latter overriding the former) and produce a 
> single HBase configuration file to install on the cluster.

--
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

        

Reply via email to