DanielLeens commented on PR #11800:
URL: https://github.com/apache/seatunnel/pull/11800#issuecomment-5558558135

   Thanks @davidzollo!
   
   For the record on my side: no code/doc-correctness blockers remain at this 
head (`da586b145d39`, unchanged since my 2026-09-01 review) — the Maxcompute 
column-projection checklist fix and the zh Kafka stray-fence removal both 
re-verified accurate against source, and the fence-tagging/`batch_interval_ms` 
issues from my first-round review were already resolved by a full revert back 
to `dev`.
   
   The one item still outstanding from my last review is non-blocking but 
hasn't been addressed yet: the PR description still describes the old, much 
larger scope (16 files across IoTDB/Kafka/Maxcompute/OceanBase/Oracle, 
including the fabricated `batch_interval_ms` option), while the actual current 
diff only touches the Maxcompute checklist and one zh Kafka file. Since this PR 
is specifically about docs accuracy, it'd be good to tidy the description to 
match before merge, but I'm not holding up the merge on it.


-- 
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]

Reply via email to