[GitHub] keith-turner commented on a change in pull request #132: Refactored configuration overview

[GitBox <gi...@apache.org>]

keith-turner commented on a change in pull request #132: Refactored configuration overview
URL: https://github.com/apache/accumulo-website/pull/132#discussion_r234632014
 
 

 ##########
 File path: _docs-2/configuration/overview.md
 ##########
 @@ -13,61 +13,72 @@ Configuration is managed differently for Accumulo clients and servers.
 
 ## Server Configuration
 
-Accumulo services (i.e master, tablet server, monitor, etc) are configured by [server properties] whose values can be
-set in the following locations (with increasing precedence):
+Accumulo processes (i.e master, tablet server, monitor, etc) are configured by [server properties] whose values can be
+set in the following configuration locations (with increasing precedence) that are described in detail below:
 
-1. Default values
-2. [accumulo.properties] (overrides defaults)
-3. Zookeeper (overrides [accumulo.properties] & defaults)
+1. [Default](#default) - All properties have a default value
+2. [Site](#site) - Properties set in [accumulo.properties]
+3. [System](#system) - Properties set using shell or Java API that apply to entire Accumulo instance
+4. [Namespace](#namespace) - Table properties set using shell or Java API that apply to a table namespace
+5. [Table](#table) - Table properties set using shell or Java API that apply to a table.
 
-If a property is set in multiple locations, the value in the location with the highest precedence is used. 
+If a property is set in multiple locations, the value in the location with the highest precedence is used.
 
-The configuration locations above are described in detail below.
+These configuration locations are described in detail below:
 
-### Default values
+### Default
 
 All [server properties] have a default value. Default values are set in the source code and can be viewed for each property on the [server properties] page.
 While default values have the lowest precedence, they are usually optimal.  However, there are cases where a change can increase query and ingest performance.
 
-### accumulo.properties
+### Site
 
-Setting [server properties] in [accumulo.properties] will override their default value. If you are running Accumulo on a cluster, any updates to accumulo.properties must
-be synced across the cluster. Accumulo processes (master, tserver, etc) read their local [accumulo.properties] on start up so processes must be restarted to apply changes.
-Certain properties can only be set in accumulo.properties. These properties have **zk mutable: no** in their description. Setting properties in accumulo.properties allows you
-to configure tablet servers with different settings.
+Site configuration refers to [server properties] set in the [accumulo.properties] file. Site configuration will override the default value of a property. If you are
 
 Review comment:
   Could mention where this file might be found.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services