Oscarcheng0312 commented on code in PR #17092: URL: https://github.com/apache/dolphinscheduler/pull/17092#discussion_r2019198158
########## docs/docs/en/architecture/design.md: ########## @@ -59,21 +59,25 @@ - **RetryReportTaskStatusThread** is mainly responsible for regularly polling to report the task status to the Master until the Master replies to the status ack to avoid the loss of the task status; -* **ZooKeeper** +- **ZooKeeper** - ZooKeeper service, MasterServer and WorkerServer nodes in the system all use ZooKeeper for cluster management and fault tolerance. In addition, the system implements event monitoring and distributed locks based on ZooKeeper. + ZooKeeper service, MasterServer and WorkerServer nodes in the system all use ZooKeeper for cluster management and fault tolerance. With evolving needs and modern deployment environments, DolphinScheduler now supports event monitoring and distributed locks not only based on ZooKeeper, but also on **JDBC** and **Etcd** implementations. + + - **JDBC** provides a lightweight solution for lock and event management through relational databases, making it ideal for simpler deployments or systems that already rely heavily on SQL databases. + + - **Etcd** enables distributed coordination using a high-performance key-value store, suitable for modern cloud-native environments. Review Comment: Thank you very much for your feedback, and I'm really sorry for the confusion. This is my very first PR, and I misunderstood the scope of this issue. I thought the goal was to add a basic description of the newly supported JDBC and Etcd implementations. I will now revise the PR to focus more on how DolphinScheduler technically supports JDBC/Etcd for event monitoring and distributed locking. Thank you again for your patience and guidance! -- 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]
