adoroszlai commented on a change in pull request #2050:
URL: https://github.com/apache/ozone/pull/2050#discussion_r595769572
##########
File path: hadoop-hdds/docs/content/feature/OM-HA.md
##########
@@ -27,15 +27,15 @@ Ozone has two leader nodes (*Ozone Manager* for key space
management and *Storag
To avoid any single point of failure the leader nodes also should have a HA
setup.
- 1. HA of Ozone Manager is implemented with the help of RAFT (Apache Ratis)
- 2. HA of Storage Container Manager is [under implementation]({{< ref
"scmha.md">}})
+Both Ozone Manager and Storage Container Manager supports HA. In this mode the
internal state is replicated via RAFT (with Apache Ratis)
+
+This document explain the HA setup of Ozone Manager (OM) HA, please check
[this page[({{< ref "SCM-HA" >}})]. While they can be setup for HA
independently, a reliable, full HA setup requires enabling HA for both services.
Review comment:
No, it's not valid. It renders as:
```
please check [this page[(/feature/scm-ha.html)].
```
```suggestion
This document explain the HA setup of Ozone Manager (OM) HA, please check
[this page]({{< ref "SCM-HA" >}}) for SCM HA. While they can be setup for HA
independently, a reliable, full HA setup requires enabling HA for both services.
```
Steps to verify:
```
mvn -pl :hadoop-hdds-docs clean package
open hadoop-hdds/docs/target/classes/docs/feature/om-ha.html
```
----------------------------------------------------------------
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
