HIVE-29346: Optimize the HMS Docker image and enable S3 support #6215
+298
−81
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
deniskuzZ
commented
Nov 26, 2025
| ); | ||
| // Use UTC date to ensure reader date is same on all timezones. | ||
| LocalDate expiredDate | ||
| = LocalDateTime.ofEpochSecond((clock.getTime() - ttl) / 1000, 0, ZoneOffset.UTC).toLocalDate(); |
Member
Author
There was a problem hiding this comment.
removes YARN dependency in HMS
deniskuzZ
commented
Nov 26, 2025
| RUN apt-get update && \ | ||
| apt-get install -y wget | ||
|
|
||
| RUN tar -xzv \ |
Member
Author
There was a problem hiding this comment.
@ayushtkn anything else we could cut?
du -h /opt/hadoop
481M
418M ./share/hadoop/common/lib
423M ./share/hadoop/common
standalone-metastore/packaging/src/docker/conf/metastore-site.xml
Outdated
Show resolved
Hide resolved
deniskuzZ
force-pushed
the
HMS_DOCKER
branch
from
1036e2f to
0bdaf42
Compare
deniskuzZ
force-pushed
the
HMS_DOCKER
branch
2 times, most recently
from
70e6500 to
1e8fa6e
Compare
deniskuzZ
force-pushed
the
HMS_DOCKER
branch
from
1e8fa6e to
e7f20bf
Compare
deniskuzZ
force-pushed
the
HMS_DOCKER
branch
3 times, most recently
from
ee72c4c to
77e2add
Compare
deniskuzZ
force-pushed
the
HMS_DOCKER
branch
from
77e2add to
bd1e47a
Compare
deniskuzZ
force-pushed
the
HMS_DOCKER
branch
2 times, most recently
from
c68cd2e to
88ede49
Compare
deniskuzZ
force-pushed
the
HMS_DOCKER
branch
2 times, most recently
from
708f7e6 to
021da73
Compare
deniskuzZ
changed the title
deniskuzZ
force-pushed
the
HMS_DOCKER
branch
from
021da73 to
a8346f2
Compare
Contributor
|
LGTM +1 (non-binding)
2. S3A configs looks good ✅
3. Was able to build docker images using `-Pdocker` profile
|
Aggarwal-Raghav
approved these changes
Nov 27, 2025
asf-ci-hive
added
tests pending
tests unstable
and removed
tests passed
tests pending
tests unstable
labels
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What changes were proposed in this pull request?
Why are the changes needed?
Create production ready docker image
Does this PR introduce any user-facing change?
No
How was this patch tested?
Locally