wombatu-kun commented on PR #19202: URL: https://github.com/apache/hudi/pull/19202#issuecomment-4954666893
@danny0405 @cshuo could I get a review here? This is the CDC streaming-read teardown race, and it is now failing CI on unrelated PRs (second sighting linked above, on a PR that only changes a release shell script). Tagging you two because it sits on the path you own: the race only became reachable by default once native CDC read (#19114) landed and #19118 switched the default log format to native, so `BaseImageIterator` is now on the hot path of the stream-read ITs. The change is small: `CdcFileSplitsIterator.hasNext()/next()/close()` become `synchronized` with a `closed` flag, which also covers the nested per-split iterators since they are only ever reached through that instance. Branch is green (73/73 including Azure) and mergeable. -- 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]
