abstractdog commented on code in PR #6435: URL: https://github.com/apache/hive/pull/6435#discussion_r3129736268
########## packaging/src/docker/build.sh: ########## @@ -20,13 +20,15 @@ set -eux HIVE_VERSION= HADOOP_VERSION= TEZ_VERSION= +TEZ_SNAPSHOT_VERSION= Review Comment: I'm open to splitting this into 2 PRs regarding tez base image: yeah, theoretically, hive image can depend upon the Tez one, but I still feel that in order to be able to move fast later, hive has to build its standalone TezAM image without actually depending on the Tez image, similarly to Tez doesn't depend on the Hadoop, and also Hive image so far doesn't depend on the Hadoop image (https://hub.docker.com/r/apache/hadoop/tags) -- 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]
