DanielLeens commented on PR #10808: URL: https://github.com/apache/seatunnel/pull/10808#issuecomment-4924192084
Thanks @SEZ9, I agree the docs need one more pass before merge. In particular, the new `nodeRole`, `coordinator`, and `worker` fields should not live only in the sample JSON; the EN/ZH REST docs should describe the valid `nodeRole` values and make the `isMaster` vs capability distinction explicit for separated deployments. From Daniel's side I still do not see a reopened runtime blocker in the current code path, but I would treat the missing field-level docs as something to close before merge on this head. -- 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]
