ctubbsii commented on code in PR #409:
URL: https://github.com/apache/accumulo-website/pull/409#discussion_r1430751551


##########
_docs-2/configuration/server-properties.md:
##########
@@ -119,6 +119,7 @@ Below are properties set in `accumulo.properties` or the 
Accumulo shell that con
 | <a name="monitor_prefix" class="prop"></a> **monitor.*** | **Available 
since:** 1.3.5<br>Properties in this category affect the behavior of the 
monitor web server. |
 | <a name="monitor_lock_check_interval" class="prop"></a> 
monitor.lock.check.interval | **Available since:** 1.5.1<br>The amount of time 
to sleep between checking for the Monitor ZooKeeper lock<br>**type:** 
TIMEDURATION, **zk mutable:** no, **default value:** `5s` |
 | <a name="monitor_port_client" class="prop"></a> monitor.port.client | 
**Available since:** 1.3.5<br>The listening port for the monitor's http 
service<br>**type:** PORT, **zk mutable:** no, **default value:** `9995` |
+| <a name="monitor_port_log4j" class="prop"></a> monitor.port.log4j | 
**Available since:** 1.3.5<br>The listening port for the monitor's http 
service<br>**type:** PORT, **zk mutable:** no, **default value:** `9995` |

Review Comment:
   This is a generated file and should not be manually modified. Also, this 
property does not exist for 2.x or higher, so should not be added to the docs 
intended for 2.x and higher. The entry in the network table should just be 
removed, as it's not valid. The changes to this file can be reverted.
   
   Everything else looks good, though.



-- 
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: notifications-unsubscr...@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to