You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Grant Henke (Code Review)" <ge...@cloudera.org> on 2020/10/13 18:57:08 UTC
[kudu-CR] [subprocess] Avoid transitive log4j dependencies
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/16592
Change subject: [subprocess] Avoid transitive log4j dependencies
......................................................................
[subprocess] Avoid transitive log4j dependencies
This patch excludes the log4j and slf4j dependencies
from the Hadoop and Ranger dependencies to
ensure we use only our explicitly defined versions.
This patch also adds a test log4j2.properties file
that was missing.
Change-Id: I48db5ee55e0f8d9f38b0351b2b3f7509175f6320
---
M java/kudu-subprocess/build.gradle
A java/kudu-subprocess/src/test/resources/log4j2.properties
2 files changed, 39 insertions(+), 4 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/92/16592/1
--
To view, visit http://gerrit.cloudera.org:8080/16592
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I48db5ee55e0f8d9f38b0351b2b3f7509175f6320
Gerrit-Change-Number: 16592
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <gr...@apache.org>
[kudu-CR] [subprocess] Avoid transitive log4j dependencies
Posted by "Andrew Wong (Code Review)" <ge...@cloudera.org>.
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16592 )
Change subject: [subprocess] Avoid transitive log4j dependencies
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/16592
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I48db5ee55e0f8d9f38b0351b2b3f7509175f6320
Gerrit-Change-Number: 16592
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Tue, 13 Oct 2020 19:36:55 +0000
Gerrit-HasComments: No
[kudu-CR] [subprocess] Avoid transitive log4j dependencies
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16592 )
Change subject: [subprocess] Avoid transitive log4j dependencies
......................................................................
[subprocess] Avoid transitive log4j dependencies
This patch excludes the log4j and slf4j dependencies
from the Hadoop and Ranger dependencies to
ensure we use only our explicitly defined versions.
This patch also adds a test log4j2.properties file
that was missing.
Change-Id: I48db5ee55e0f8d9f38b0351b2b3f7509175f6320
Reviewed-on: http://gerrit.cloudera.org:8080/16592
Tested-by: Kudu Jenkins
Reviewed-by: Andrew Wong <aw...@cloudera.com>
---
M java/kudu-subprocess/build.gradle
A java/kudu-subprocess/src/test/resources/log4j2.properties
2 files changed, 39 insertions(+), 4 deletions(-)
Approvals:
Kudu Jenkins: Verified
Andrew Wong: Looks good to me, approved
--
To view, visit http://gerrit.cloudera.org:8080/16592
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I48db5ee55e0f8d9f38b0351b2b3f7509175f6320
Gerrit-Change-Number: 16592
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)