abstractdog commented on code in PR #6435:
URL: https://github.com/apache/hive/pull/6435#discussion_r3100300911
##########
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}
+ container_name: tezam
+ hostname: tezam
+ depends_on:
+ - zookeeper
+ restart: on-failure:3
+ environment:
+ USER: hive
+ SERVICE_NAME: 'tezam'
+
+ TEZ_FRAMEWORK_MODE: STANDALONE_ZOOKEEPER
+ TEZ_AM_ZOOKEEPER_QUORUM: zookeeper:2181
+
+ # LLAP daemon discovery
+ HIVE_ZOOKEEPER_QUORUM: zookeeper:2181
+ LLAP_SERVICE_HOSTS: '@llap0'
Review Comment:
good catch, it's not correct, the only reason why it works is that I take
care of it one level deeper in entrypoint:
https://github.com/apache/hive/pull/6435/changes#diff-b7d5fbeab2c6af92616ab371fa3f237867d60d7c81072045342cb44a8981bf90R45
fixing this
--
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]
