abstractdog commented on code in PR #6435:
URL: https://github.com/apache/hive/pull/6435#discussion_r3099756277
##########
packaging/src/docker/docker-compose.yml:
##########
@@ -124,14 +124,35 @@ services:
- zookeeper_datalog:/datalog
- zookeeper_logs:/logs
- #TODO Tez AM container (in the meantime, the HS2(with local Tez AM) + LLAP
daemon setup is working properly)
- # 1. Define and use a Tez AM image from HIVE-29419 or TEZ-4682
- # 2. Configure TezAM to use Zookeeper Llap Registry to discover the LLAP
daemon
- # 3. Configure HiveServer2 to use the Tez AM Zookeeper Registry to
discover the Tez AM
- # Prerequisites:
- # - tez-api 1.0.0-SNAPSHOT jar injected into HiveSever2 until Tez
1.0.0 is released
- # - make HIVE-29477 happen to let HiveServer2 use Tez external sessions
- # 4. Define hadoop components here to be used by all the containers
(working example can be found at TEZ-4682), currently a local volume
+ tezam:
+ profiles:
+ - llap
+ image: apache/hive:${HIVE_VERSION}
Review Comment:
I think it's ok to reuse, actually, we build both (Hive specific TezAM image
+ Tez AM image in Tez project), see the motivation on
[HIVE-29419](https://issues.apache.org/jira/browse/HIVE-29419)
downstream we have only a hive image for TezAM, but upstream, the Tez
project needs to have its own,
[TEZ-4682](https://issues.apache.org/jira/browse/TEZ-4682), most probably
mimics the Tez Container mode (DAGAppMaster + TezChiild), not the LLAP
--
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]
