zhangshenghang commented on PR #12932: URL: https://github.com/apache/seatunnel/pull/12932#issuecomment-6313414227
All six review points are addressed in d621e7dbc: 1. **Issue 1 (blocker)**: the unverified 'copy or move the savepoint directories' workaround is removed from 6.3 (en+zh); the section now states that `restoreMode=SAVEPOINT` always picks the savepoint with the highest checkpoint id per pipeline and that the REST API offers no supported way to select an older one. 2. **Issue 2**: the 5.2 resume example now points to section 6.2 instead of the undefined `fake_to_console.conf`. 3. **Issue 3**: the FAQ keyless-table answer now leads with 'Yes, with limitations.' and the identical-rows caveat; OpenGauss CDC is added to the JDBC CDC connector list (en+zh). 4. **Issue 4**: the savepoint-path sentence now says the location is in the job and checkpoint logs and checkpoint storage. 5. **Issue 5**: the FakeSource `valueConstraints` sample is `'row.num' >= 0` in all four REST docs (v1/v2, en/zh). 6. **Issue 6**: both v1 log-list sections now note the endpoints require `seatunnel.engine.http.enable-http: true` (default `false`). -- 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]
