j1wonpark commented on PR #4091: URL: https://github.com/apache/amoro/pull/4091#issuecomment-3925744847
FYI — #4098 by @baiyangtx also addresses this same Docker 29+ issue with a cleaner long-term approach (centralizing all Testcontainers versions under a single parent property). This PR takes a more conservative workaround (`docker-java.properties` + partial upgrade) to unblock CI quickly while preserving Flink module compatibility. If #4098 is the preferred direction, I'm happy to close this one in favor of it. Just in case the comparison is helpful: | | This PR (#4091) | #4098 | |---|---|---| | Approach | `docker-java.properties` + upgrade to 1.20.1 | Centralize to 1.21.3 via parent property | | Flink modules | Keeps original versions (avoids `ContainerDef` clash) | Unified version (may need compat check) | | CI | Passing | Dependency list needs update | -- 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]
