Skip to content

HIVE-29686: Env variables for substitution are not propagated to entrypoint.sh from start-hive.sh#6566

Open
abstractdog wants to merge 2 commits into
apache:masterfrom
abstractdog:HIVE-29686
Open

HIVE-29686: Env variables for substitution are not propagated to entrypoint.sh from start-hive.sh#6566
abstractdog wants to merge 2 commits into
apache:masterfrom
abstractdog:HIVE-29686

Conversation

@abstractdog

@abstractdog abstractdog commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Add env variables to hiveserver2 configuration in the docker-compose file.

Why are the changes needed?

See jira description.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Build and start hive cluster according to README under packaging/src/docker. Without the patch, the queries still passed flawlessly and silently, but never reached tezam and llapdaemons. With the patch, DAG activity was present:

docker compose logs -f tezam
...

tezam  | 2026-06-26T10:27:11,512 INFO  DAGAppMaster - Completed cleanup for DAG: name=insert into test_part values (1, '1') (Stage-1), with id=dag_1782468450563_0003_2

@ayushtkn ayushtkn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanx @abstractdog for the fix, I deployed and checked the TezAM logs & it is working as expected

Image

One additional stuff, I think we should check the readme it says there is --llap option in stop but it doesn't work that way, we can update that

```shell
./stop-hive.sh --llap # to stop and delete containers
#OR
./stop-hive.sh --llap --cleanup # to remove volumes also
```

ayushsaxena@Q3NW54Y0C5 docker % ./stop-hive.sh --llap --cleanup
Unknown option: --llap
ayushsaxena@Q3NW54Y0C5 docker % ./stop-hive.sh --llap          
Unknown option: --llap
ayushsaxena@Q3NW54Y0C5 docker % 

@abstractdog

abstractdog commented Jun 26, 2026

Copy link
Copy Markdown
Contributor Author

Thanx @abstractdog for the fix, I deployed and checked the TezAM logs & it is working as expected

Image One additional stuff, I think we should check the readme it says there is `--llap` option in stop but it doesn't work that way, we can update that

```shell
./stop-hive.sh --llap # to stop and delete containers
#OR
./stop-hive.sh --llap --cleanup # to remove volumes also
```

ayushsaxena@Q3NW54Y0C5 docker % ./stop-hive.sh --llap --cleanup
Unknown option: --llap
ayushsaxena@Q3NW54Y0C5 docker % ./stop-hive.sh --llap          
Unknown option: --llap
ayushsaxena@Q3NW54Y0C5 docker % 

right! bumped into the same, but forgot to fix, let me take care of it in this PR

UPDATE: fixed in 02b40c6

@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants