Skip to content

Commit 7a3d0ca

Browse files
Dilli-Babu-Godarigodaridillibabu
authored and
godaridillibabu
committed
Refactor log4j-core and log4j-api dependencies out of root POM
refactor of PR: #24507 Upgrading module specific dependencies. Upgraded org.apache.logging.log4j:log4j-core from 2.17.1 to 2.24.3 Upgraded org.apache.logging.log4j:log4j-api from 2.17.1 to 2.24.3
1 parent 15afc5c commit 7a3d0ca

File tree

3 files changed

+6
-16
lines changed

3 files changed

+6
-16
lines changed

pom.xml

-16
Original file line numberDiff line numberDiff line change
@@ -103,8 +103,6 @@
103103
<air.test.thread-count>2</air.test.thread-count>
104104
<air.test.jvmsize>4g</air.test.jvmsize>
105105
<grpc.version>1.68.0</grpc.version>
106-
<dep.log4j.version>2.24.3</dep.log4j.version>
107-
108106
<air.javadoc.lint>-missing</air.javadoc.lint>
109107
</properties>
110108

@@ -234,20 +232,6 @@
234232
<scope>import</scope>
235233
</dependency>
236234

237-
<dependency>
238-
<groupId>org.apache.logging.log4j</groupId>
239-
<artifactId>log4j-api</artifactId>
240-
<version>${dep.log4j.version}</version>
241-
<scope>runtime</scope>
242-
</dependency>
243-
244-
<dependency>
245-
<groupId>org.apache.logging.log4j</groupId>
246-
<artifactId>log4j-core</artifactId>
247-
<version>${dep.log4j.version}</version>
248-
<scope>runtime</scope>
249-
</dependency>
250-
251235
<dependency>
252236
<groupId>com.facebook.presto</groupId>
253237
<artifactId>presto-testing-docker</artifactId>

presto-druid/pom.xml

+3
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313

1414
<properties>
1515
<air.main.basedir>${project.parent.basedir}</air.main.basedir>
16+
<dep.log4j.version>2.24.3</dep.log4j.version>
1617
</properties>
1718

1819
<repositories>
@@ -177,12 +178,14 @@
177178
<dependency>
178179
<groupId>org.apache.logging.log4j</groupId>
179180
<artifactId>log4j-api</artifactId>
181+
<version>${dep.log4j.version}</version>
180182
<scope>runtime</scope>
181183
</dependency>
182184

183185
<dependency>
184186
<groupId>org.apache.logging.log4j</groupId>
185187
<artifactId>log4j-core</artifactId>
188+
<version>${dep.log4j.version}</version>
186189
<scope>runtime</scope>
187190
</dependency>
188191

presto-elasticsearch/pom.xml

+3
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
<properties>
1414
<air.main.basedir>${project.parent.basedir}</air.main.basedir>
1515
<dep.elasticsearch.version>6.0.0</dep.elasticsearch.version>
16+
<dep.log4j.version>2.24.3</dep.log4j.version>
1617
</properties>
1718

1819
<dependencies>
@@ -172,12 +173,14 @@
172173
<dependency>
173174
<groupId>org.apache.logging.log4j</groupId>
174175
<artifactId>log4j-api</artifactId>
176+
<version>${dep.log4j.version}</version>
175177
<scope>runtime</scope>
176178
</dependency>
177179

178180
<dependency>
179181
<groupId>org.apache.logging.log4j</groupId>
180182
<artifactId>log4j-core</artifactId>
183+
<version>${dep.log4j.version}</version>
181184
<scope>runtime</scope>
182185
</dependency>
183186

0 commit comments

Comments
 (0)