DomGarguilo commented on a change in pull request #300:
URL: https://github.com/apache/accumulo-website/pull/300#discussion_r738604276



##########
File path: _docs-2/getting-started/quickstart.md
##########
@@ -202,25 +202,30 @@ from provided templates:
 accumulo-cluster create-config
 ```
 
-This creates five files ([managers], [gc], [monitor], [tservers], & [tracers]) 
in
-the `conf/` directory that contain the node names where Accumulo services are
-run on your cluster. By default, all files are configured to `localhost`. If 
you
+This creates a yaml configuration file in the `conf/` directory named
+`cluster.yaml` that contains the node names where Accumulo services are
+run on your cluster. By default, all services are configured to `localhost`. 
If you
 are running a single-node Accumulo cluster, these files do not need to be
-changed and the next section should be skipped.
+changed and the next section should be skipped. The external compaction 
services
+exist in the file but are commented out as they are optional.
 
 #### Multi-node configuration
 
-If you are running an Accumulo cluster on multiple nodes, the following files 
in
-`conf/` should be configured with a newline separated list of node names:
+If you are running an Accumulo cluster on multiple nodes, the 
`conf/cluster.yaml`
+files contains section that should be configured with a list of node names in 
yaml format:
 
- * [managers] : Accumulo primary coordinating process. Must specify one node. 
Can
+ * [manager] : Accumulo primary coordinating process. Must specify one node. 
Can
    specify a few for fault tolerance.
  * [gc]      : Accumulo garbage collector. Must specify one node. Can specify a
    few for fault tolerance.
  * [monitor] : Node where Accumulo monitoring web server is run.
- * [tservers] : Accumulo worker processes. List all of the nodes where tablet
-   servers should run in this file.
- * [tracers] : Optional capability. Can specify zero or more nodes.
+ * [tserver] : Accumulo worker processes. List all of the nodes where tablet
+   servers should run.
+ * [tracer] : Optional capability. Can specify zero or more nodes.
+ * [compaction.coordinator] : Optional. Must specify one node. Can specify a 
few
+   for fault tolerance.
+ * [compaction.compactor] : Optional. Accumulo external compactor processes. 
List of
+   all nodes where compactors should run

Review comment:
       ```suggestion
      all nodes where compactors should run.
   ```

##########
File path: _docs-2/configuration/files.md
##########
@@ -51,18 +51,18 @@ The {% ghc assemble/conf/log4j-monitor.properties %} file 
configures logging for
 The {% ghc assemble/conf/log4j.properties %} file configures logging for 
Accumulo commands (i.e `accumulo init`,
 `accumulo shell`, etc).
 
-## Host files
+## cluster.yaml
 
-The `accumulo-cluster` script uses the host files below to determine where 
Accumulo processes should be run.
-These files are not in the `conf/` directory of the Accumulo release tarball 
by default. They can be created by running
-the command `accumulo-cluster create-config`.
+The `accumulo-cluster` script uses the `cluster.yaml` file to determine where 
Accumulo processes should be run.
+This file is not in the `conf/` directory of the Accumulo release tarball by 
default. It can be created by running
+the command `accumulo-cluster create-config`. The `clustery.yaml` file 
contains the following sections:

Review comment:
       ```suggestion
   the command `accumulo-cluster create-config`. The `cluster.yaml` file 
contains the following sections:
   ```

##########
File path: _docs-2/configuration/files.md
##########
@@ -72,19 +72,31 @@ to run on standby Managers that can take over if the lead 
Manager fails.
 Contains a list of hosts where [Monitor] processes should run. While only one 
host is needed, others can be specified
 to run standby Monitors that can take over if the lead Monitor fails.
 
-### tservers
+### tserver
 
 Contains list of hosts where [Tablet Server] processes should run. While only 
one host is needed, it is recommended that
 multiple tablet servers are run for improved fault tolerance and peformance.
 
-### tracers
+### tracer
 
 Contains a list of hosts where [Tracer] processes should run. While only one 
host is needed, others can be specified
 to run standby Tracers that can take over if the lead Tracer fails.
 
+### Compaction Coordinator
+
+Contains a list of hosts where [CompactionCoordinator] processes should run. 
While only one host is needed,
+others can be specified to run standby CompactionCoordinators that can take 
over if the lead CompactionCoordinator fails.
+
+### Compactor
+
+Contains a list of hosts where [Compactor] processes should run. While only 
one host is needed, it is recommended that
+multiple Compactors are run for improved external compaction performace.

Review comment:
       ```suggestion
   multiple Compactors are run for improved external compaction performance.
   ```

##########
File path: _docs-2/getting-started/quickstart.md
##########
@@ -202,25 +202,30 @@ from provided templates:
 accumulo-cluster create-config
 ```
 
-This creates five files ([managers], [gc], [monitor], [tservers], & [tracers]) 
in
-the `conf/` directory that contain the node names where Accumulo services are
-run on your cluster. By default, all files are configured to `localhost`. If 
you
+This creates a yaml configuration file in the `conf/` directory named
+`cluster.yaml` that contains the node names where Accumulo services are
+run on your cluster. By default, all services are configured to `localhost`. 
If you
 are running a single-node Accumulo cluster, these files do not need to be
-changed and the next section should be skipped.
+changed and the next section should be skipped. The external compaction 
services
+exist in the file but are commented out as they are optional.
 
 #### Multi-node configuration
 
-If you are running an Accumulo cluster on multiple nodes, the following files 
in
-`conf/` should be configured with a newline separated list of node names:
+If you are running an Accumulo cluster on multiple nodes, the 
`conf/cluster.yaml`
+files contains section that should be configured with a list of node names in 
yaml format:

Review comment:
       ```suggestion
   file contains sections that should be configured with a list of node names 
in yaml format:
   ```




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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to