[accumulo-testing] branch main updated: Fix logging bug and update hadoop version (#225)

[do...@apache.org]

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

domgarguilo pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/accumulo-testing.git


The following commit(s) were added to refs/heads/main by this push:
     new 5b12805  Fix logging bug and update hadoop version (#225)
5b12805 is described below

commit 5b12805f8db7023e2daca033b07cb99b7f8bbec8
Author: Dom G <do...@apache.org>
AuthorDate: Wed Sep 21 16:10:17 2022 -0400

    Fix logging bug and update hadoop version (#225)
    
    * update log4j/slf4j version
    * update hadoop version
---
 contrib/terraform-testing-infrastructure/README.md          | 4 ++--
 contrib/terraform-testing-infrastructure/aws/variables.tf   | 2 +-
 contrib/terraform-testing-infrastructure/azure/variables.tf | 2 +-
 pom.xml                                                     | 6 +++++-
 4 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/contrib/terraform-testing-infrastructure/README.md b/contrib/terraform-testing-infrastructure/README.md
index e86a7a5..d36bcad 100644
--- a/contrib/terraform-testing-infrastructure/README.md
+++ b/contrib/terraform-testing-infrastructure/README.md
@@ -163,7 +163,7 @@ The table below lists the variables and their default values that are used in th
 | cloudinit\_merge\_type | Describes the merge behavior for overlapping config blocks in cloud-init. | `string` | `null` | no |
 | create\_route53\_records | Indicates whether or not route53 records will be created | `bool` | `false` | no |
 | hadoop\_dir | The Hadoop directory on each EC2 node | `string` | `"/data/hadoop"` | no |
-| hadoop\_version | The version of Hadoop to download and install | `string` | `"3.3.3"` | no |
+| hadoop\_version | The version of Hadoop to download and install | `string` | `"3.3.4"` | no |
 | instance\_count | The number of EC2 instances to create | `string` | `"2"` | no |
 | instance\_type | The type of EC2 instances to create | `string` | `"m5.2xlarge"` | no |
 | local\_sources\_dir | Directory on local machine that contains Maven, ZooKeeper or Hadoop binary distributions or Accumulo source tarball | `string` | `""` | no |
@@ -207,7 +207,7 @@ The table below lists the variables and their default values that are used in th
 | cloudinit\_merge\_type | Describes the merge behavior for overlapping config blocks in cloud-init. | `string` | `null` | no |
 | create\_resource\_group | Indicates whether or not resource\_group\_name should be created or is an existing resource group. | `bool` | `true` | no |
 | hadoop\_dir | The Hadoop directory on each node | `string` | `"/data/hadoop"` | no |
-| hadoop\_version | The version of Hadoop to download and install | `string` | `"3.3.3"` | no |
+| hadoop\_version | The version of Hadoop to download and install | `string` | `"3.3.4"` | no |
 | local\_sources\_dir | Directory on local machine that contains Maven, ZooKeeper or Hadoop binary distributions or Accumulo source tarball | `string` | `""` | no |
 | location | The Azure region where resources are to be created. If an existing resource group is specified, this value is ignored and the resource group's location is used. | `string` | n/a | yes |
 | managed\_disk\_configuration | Optional managed disk configuration. If supplied, the managed disks on each VM will be combined into an LVM volume mounted at the named mount point. | <pre>object({<br>    mount_point          = string<br>    disk_count           = number<br>    storage_account_type = string<br>    disk_size_gb         = number<br>  })</pre> | `null` | no |
diff --git a/contrib/terraform-testing-infrastructure/aws/variables.tf b/contrib/terraform-testing-infrastructure/aws/variables.tf
index 1cddf8e..b2918f7 100644
--- a/contrib/terraform-testing-infrastructure/aws/variables.tf
+++ b/contrib/terraform-testing-infrastructure/aws/variables.tf
@@ -141,7 +141,7 @@ variable "zookeeper_version" {
 }
 
 variable "hadoop_version" {
-  default     = "3.3.3"
+  default     = "3.3.4"
   description = "The version of Hadoop to download and install"
   nullable    = false
 }
diff --git a/contrib/terraform-testing-infrastructure/azure/variables.tf b/contrib/terraform-testing-infrastructure/azure/variables.tf
index 3bfea2e..6c18fbb 100644
--- a/contrib/terraform-testing-infrastructure/azure/variables.tf
+++ b/contrib/terraform-testing-infrastructure/azure/variables.tf
@@ -219,7 +219,7 @@ variable "zookeeper_version" {
 }
 
 variable "hadoop_version" {
-  default     = "3.3.3"
+  default     = "3.3.4"
   description = "The version of Hadoop to download and install"
   nullable    = false
 }
diff --git a/pom.xml b/pom.xml
index 9b576b3..1237cb6 100644
--- a/pom.xml
+++ b/pom.xml
@@ -89,7 +89,11 @@
     </dependency>
     <dependency>
       <groupId>org.apache.logging.log4j</groupId>
-      <artifactId>log4j-slf4j-impl</artifactId>
+      <artifactId>log4j-core</artifactId>
+    </dependency>
+    <dependency>
+      <groupId>org.apache.logging.log4j</groupId>
+      <artifactId>log4j-slf4j2-impl</artifactId>
     </dependency>
     <dependency>
       <groupId>org.slf4j</groupId>