DanielLeens commented on PR #13026: URL: https://github.com/apache/seatunnel/pull/13026#issuecomment-6574875583
Follow-up on head `6b167563b80439110a76a1cb6cd65ad069ea1d3a` (unchanged since my last review). My previous review listed no source findings; the only open items were the two failed CI shards (an HTTP 502 while pulling a Docker image in `all-connectors-it-7`, and a Maven Wrapper download failure in `all-connectors-it-4`), which did not point at the Avro change. The current-head [Build check](https://github.com/apache/seatunnel/runs/124978998016) now reports SUCCESS, so that CI fact in my last review is stale and the shards were indeed environmental. With the source review clean and CI green, I have no remaining technical concern on this PR. The only gate left is the repository's maintainer approval, which a write-capable committer needs to provide. Thanks @goutamadwant for the careful work and the thorough regressions. -- 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]
