karthick-rn commented on a change in pull request #284: Support (optional) HA capabilities for Hadoop & Accumulo URL: https://github.com/apache/fluo-muchos/pull/284#discussion_r326689885
########## File path: README.md ########## @@ -280,6 +280,19 @@ master, etc. It also has variables in the `[all:vars]` section that contain sett useful in user playbooks. It is recommended that any user-defined Ansible playbooks should be managed in their own git repository (see [mikewalch/muchos-custom][mc] for an example). +## High-Availability (optional) + +Additionally, Muchos can be configured to provide High-Availability for HDFS & Accumulo components. By default, +this feature is off, however it can be turned on by editing the following settings in [muchos.props] +under the `general` section as shown below: + +```ini +hdfs_ha = True # default is False +nameservice_id = muchoshacluster # Logical name for the cluster, no special characters +``` + Review comment: > > Support for optional HA was implemented keeping in mind not to disturb the existing ‘hadoop’ role but add an additional role for ‘hadoop-ha’ making both these roles independent of each other. > > I am in favor of that goal. Also would not want to increase code complexity to dedupe templates. For the cases were things are significantly similar, I think it would make sense to add comments like. > > ``` > <!-- when editing this file please consider if changes are also needed in roles/hadoop-ha/templates --> > ``` Thanks! Makes complete sense. I'll pop this comment on the config files under `hadoop` ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
