DanielLeens commented on PR #11803: URL: https://github.com/apache/seatunnel/pull/11803#issuecomment-5341010232
Thanks for the ping, and +1 to @nzw921rx's approval as well. From my side the conclusion is unchanged since my last review: the timestamp validation now matches the runtime behavior (start_timestamp can be 0, an explicitly-configured end_timestamp must be > 0), the zero-end and `[0, 1)` edge cases are covered, and both `docs/en`/`docs/zh` were updated. CI is green and the full connector-hbase test suite passes. Nothing further from me — this looks ready to merge. -- 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]
