You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bigtop.apache.org by el...@apache.org on 2021/12/31 11:34:58 UTC

[bigtop] branch master updated: BIGTOP-3626: Upgrade log4j dependencies for ycsb (#848)

This is an automated email from the ASF dual-hosted git repository.

elukey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/bigtop.git


The following commit(s) were added to refs/heads/master by this push:
     new 265e891  BIGTOP-3626: Upgrade log4j dependencies for ycsb (#848)
265e891 is described below

commit 265e891afe1c374a7eb8199963ef20913322b994
Author: Luca Toscano <el...@users.noreply.github.com>
AuthorDate: Fri Dec 31 12:34:53 2021 +0100

    BIGTOP-3626: Upgrade log4j dependencies for ycsb (#848)
    
    At the time of writing upstream didn't release any official fix,
    but https://github.com/brianfrankcooper/YCSB/pull/1583 seems
    taking care of it.
    
    Credits for the upstream fix: Filipe Oliveira <fi...@gmail.com>
---
 bigtop-packages/src/common/ycsb/patch1-log4j.diff | 65 +++++++++++++++++++++++
 bigtop.bom                                        |  2 +-
 2 files changed, 66 insertions(+), 1 deletion(-)

diff --git a/bigtop-packages/src/common/ycsb/patch1-log4j.diff b/bigtop-packages/src/common/ycsb/patch1-log4j.diff
new file mode 100644
index 0000000..8d02b0d
--- /dev/null
+++ b/bigtop-packages/src/common/ycsb/patch1-log4j.diff
@@ -0,0 +1,65 @@
+diff --git a/elasticsearch5/pom.xml b/elasticsearch5/pom.xml
+index 5d3ff06710..f10476cf05 100644
+--- a/elasticsearch5/pom.xml
++++ b/elasticsearch5/pom.xml
+@@ -165,12 +165,12 @@ LICENSE file.
+     <dependency>
+       <groupId>org.apache.logging.log4j</groupId>
+       <artifactId>log4j-api</artifactId>
+-      <version>2.8.2</version>
++      <version>2.17.0</version>
+     </dependency>
+     <dependency>
+       <groupId>org.apache.logging.log4j</groupId>
+       <artifactId>log4j-core</artifactId>
+-      <version>2.8.2</version>
++      <version>2.17.0</version>
+     </dependency>
+     <dependency>
+       <groupId>junit</groupId>
+diff --git a/ignite/pom.xml b/ignite/pom.xml
+index eabf8d67d9..7b3ed0d496 100644
+--- a/ignite/pom.xml
++++ b/ignite/pom.xml
+@@ -87,13 +87,13 @@ LICENSE file.
+     <dependency>
+       <groupId>org.apache.logging.log4j</groupId>
+       <artifactId>log4j-api</artifactId>
+-      <version>2.11.0</version>
++      <version>2.17.0</version>
+     </dependency>
+ 
+     <dependency>
+       <groupId>org.apache.logging.log4j</groupId>
+       <artifactId>log4j-core</artifactId>
+-      <version>2.11.0</version>
++      <version>2.17.0</version>
+     </dependency>
+   </dependencies>
+ </project>
+diff --git a/voltdb/pom.xml b/voltdb/pom.xml
+index ab870853ad..6c8cbd2b74 100644
+--- a/voltdb/pom.xml
++++ b/voltdb/pom.xml
+@@ -44,17 +44,17 @@
+ 		<dependency>
+ 			<groupId>org.apache.logging.log4j</groupId>
+ 			<artifactId>log4j-api</artifactId>
+-			<version>2.7</version>
++			<version>2.17.0</version>
+ 		</dependency>
+ 		<dependency>
+ 			<groupId>org.apache.logging.log4j</groupId>
+ 			<artifactId>log4j-core</artifactId>
+-			<version>2.7</version>
++			<version>2.17.0</version>
+ 		</dependency>
+ 		<dependency>
+ 			<groupId>org.apache.logging.log4j</groupId>
+ 			<artifactId>log4j-slf4j-impl</artifactId>
+-			<version>2.7</version>
++			<version>2.17.0</version>
+ 		</dependency>
+ 		<!-- https://mvnrepository.com/artifact/org.voltdb/voltdbclient -->
+ 		<dependency>
+
diff --git a/bigtop.bom b/bigtop.bom
index b514689..c4cf4a0 100644
--- a/bigtop.bom
+++ b/bigtop.bom
@@ -301,7 +301,7 @@ bigtop {
     'ycsb' {
       name    = 'ycsb'
       relNotes = 'Yahoo! Cloud Serving Benchmark'
-      version { base = '0.17.0'; pkg = base; release = 1 }
+      version { base = '0.17.0'; pkg = base; release = 2 }
       tarball { destination = "$name-${version.base}.tar.gz"
                 source      = "${version.base}.tar.gz" }
       url     { site = "https://github.com/brianfrankcooper/YCSB/archive"