Copilot commented on code in PR #20270: URL: https://github.com/apache/druid/pull/20270#discussion_r3943572484
########## .dockerignore: ########## @@ -35,3 +35,8 @@ target *.DS_Store _site dependency-reduced-pom.xml + +# Allow the host-built distribution to pass its binary tarball into +# the Docker build while continuing to exclude all other build output. +!distribution/target/ +!distribution/target/apache-druid-*-bin.tar.gz Review Comment: `.dockerignore` currently re-includes the entire `distribution/target/` directory, not just the `*-bin.tar.gz` tarball. This can unintentionally add extra build outputs (e.g., checksums, intermediate files) to the Docker build context and make builds slower/larger than intended by the comment. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
