Skip to content

BIGTOP-3640: Upgrade hive's log4j2 version to 2.17.1 for Bigtop 3.0.x#862

Merged
iwasakims merged 2 commits intoapache:masterfrom
yoda-mon:upgrade-hive-log4j2
Feb 10, 2022
Merged

BIGTOP-3640: Upgrade hive's log4j2 version to 2.17.1 for Bigtop 3.0.x#862
iwasakims merged 2 commits intoapache:masterfrom
yoda-mon:upgrade-hive-log4j2

Conversation

@yoda-mon
Copy link
Copy Markdown
Contributor

@yoda-mon yoda-mon commented Feb 10, 2022

Description of PR

Upgrade hive's log4j2 version to 2.17.1 for Bigtop 3.0.x.
cf. Bigtop 1.5 has already been patched by
https://issues.apache.org/jira/browse/BIGTOP-3619 / #844

How was this patch tested?

./gradlew hive-pkg-ind
./gradlew docker-provisioner -Pstack="hive" -Psmoke_tests="hive"

For code changes:

  • Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'BIGTOP-3638. Your PR title ...')?
  • Make sure that newly added files do not have any licensing issues. When in doubt refer to https://www.apache.org/licenses/

Comment thread bigtop.bom Outdated
Copy link
Copy Markdown
Member

@iwasakims iwasakims left a comment

Choose a reason for hiding this comment

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

+1. I got no issue on Rocky Linux 8.

$ rpm -qlp output/hive/noarch/hive-3.1.2-3.el8.noarch.rpm | grep log4j
/etc/hive/conf.dist/beeline-log4j2.properties.template
/etc/hive/conf.dist/hive-exec-log4j2.properties.template
/etc/hive/conf.dist/hive-log4j2.properties.template
/etc/hive/conf.dist/llap-cli-log4j2.properties.template
/etc/hive/conf.dist/llap-daemon-log4j2.properties.template
/usr/lib/hive/lib/log4j-1.2-api-2.17.1.jar
/usr/lib/hive/lib/log4j-api-2.17.1.jar
/usr/lib/hive/lib/log4j-core-2.17.1.jar
/usr/lib/hive/lib/log4j-slf4j-impl-2.17.1.jar
/usr/lib/hive/lib/log4j-web-2.17.1.jar
$ ./docker-hadoop.sh \
   --create 1 \
   --image bigtop/puppet:trunk-rockylinux-8 \
   --memory 16g \
   --repo file:///bigtop-home/output \
   --disable-gpg-check \
   --stack hdfs,yarn,mapreduce,hive \
   --smoke-tests hive
...
Now testing...
:bigtop-tests:smoke-tests:hive:test (Thread[Execution worker for ':' Thread 2,5,main]) completed. Took 12.702 secs.

BUILD SUCCESSFUL in 46s
34 actionable tasks: 5 executed, 29 up-to-date

@iwasakims iwasakims merged commit 2a0f79e into apache:master Feb 10, 2022
iwasakims pushed a commit that referenced this pull request Feb 10, 2022
@iwasakims
Copy link
Copy Markdown
Member

I merged this to branch-3.0 and master. Thanks, @yoda-mon!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants