kokila-19 commented on code in PR #5927: URL: https://github.com/apache/hive/pull/5927#discussion_r2212441811
########## packaging/src/docker/Dockerfile: ########## @@ -14,14 +14,14 @@ # See the License for the specific language governing permissions and # limitations under the License. # -ARG BUILD_ENV +ARG BUILD_ENV=unarchive -FROM ubuntu as unarchive +FROM ubuntu AS unarchive ONBUILD COPY hadoop-*.tar.gz /opt ONBUILD COPY apache-hive-*-bin.tar.gz /opt ONBUILD COPY apache-tez-*-bin.tar.gz /opt -FROM ubuntu as archive +FROM ubuntu AS archive Review Comment: Proposed naming convention 1. local (instead of unarchive) : Uses local build for both Hadoop and Hive 2. release (instead of archive) : Downloads official Apache releases for both components 3. hybrid (instead of buildarchive) : Uses locally built Hive with official Hadoop release from Apache repository Do these terms provide clarity? -- 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: gitbox-unsubscr...@hive.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For additional commands, e-mail: gitbox-h...@hive.apache.org
