danielnadean commented on PR #11198: URL: https://github.com/apache/seatunnel/pull/11198#issuecomment-5154655767
I tracked the current head `7a0a56183562eebe12b89277f416a31dac06d291` and the previous `target` exclusion is still present, so this is not the earlier fix being overwritten. The current failure is still `seatunnel-ci-tools` -> `ImportClassCheckTest` in `unit-test (8, ubuntu-latest)` from fork run `srijan-singh/seatunnel/actions/runs/30726060311`, failing with `java.lang.OutOfMemoryError: GC overhead limit exceeded`. The root cause is that the test still parses checked-in generated protobuf sources and also retains all import AST nodes for the whole repository, which is enough to trip the JDK 8 unit-test lane even after excluding `target` outputs. I prepared a one-commit patch PR against the contributor branch here: https://github.com/srijan-singh/seatunnel/pull/1 It keeps the existing `target` exclusion, skips protobuf-generated sources detected by `@@protoc_insertion_point`, and stores only lightweight import metadata instead of full JavaParser import nodes. Merging that patch into `couchbase-sink-v1` (or cherry-picking `56c1eed88d`) should unblock the OOM in this lane. -- 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]
