You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by st...@apache.org on 2022/03/03 18:57:31 UTC

[hadoop] branch branch-3.3 updated: HADOOP-18150. Fix ITestAuditManagerDisabled test in S3A. (#4044)

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

stevel pushed a commit to branch branch-3.3
in repository https://gitbox.apache.org/repos/asf/hadoop.git


The following commit(s) were added to refs/heads/branch-3.3 by this push:
     new 909048d  HADOOP-18150. Fix ITestAuditManagerDisabled test in S3A. (#4044)
909048d is described below

commit 909048d87d168628ac1e19004b7d1f53863eef7b
Author: Mehakmeet Singh <me...@gmail.com>
AuthorDate: Fri Mar 4 00:14:28 2022 +0530

    HADOOP-18150. Fix ITestAuditManagerDisabled test in S3A. (#4044)
    
    Contributed by Mehakmeet Singh
    
    Change-Id: I25c10844e4ad64b1fd7af9a02018220a611c85e0
---
 .../org/apache/hadoop/fs/s3a/audit/ITestAuditManagerDisabled.java   | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/audit/ITestAuditManagerDisabled.java b/hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/audit/ITestAuditManagerDisabled.java
index 033d793..68aaef5 100644
--- a/hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/audit/ITestAuditManagerDisabled.java
+++ b/hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/audit/ITestAuditManagerDisabled.java
@@ -28,9 +28,10 @@ import org.apache.hadoop.fs.s3a.performance.AbstractS3ACostTest;
 
 import static org.apache.hadoop.fs.s3a.audit.AuditTestSupport.NOOP_SPAN;
 import static org.apache.hadoop.fs.s3a.audit.AuditTestSupport.resetAuditOptions;
+import static org.apache.hadoop.fs.s3a.audit.S3AAuditConstants.AUDIT_ENABLED;
 
 /**
- * Verify that by default audit managers are disabled.
+ * Verify that audit managers are disabled if set to false.
  */
 public class ITestAuditManagerDisabled extends AbstractS3ACostTest {
 
@@ -42,11 +43,12 @@ public class ITestAuditManagerDisabled extends AbstractS3ACostTest {
   public Configuration createConfiguration() {
     Configuration conf = super.createConfiguration();
     resetAuditOptions(conf);
+    conf.setBoolean(AUDIT_ENABLED, false);
     return conf;
   }
 
   /**
-   * The default auditor is the no-op auditor.
+   * Verify that the auditor is the no-op auditor if auditing is disabled.
    */
   @Test
   public void testAuditorDisabled() {

---------------------------------------------------------------------
To unsubscribe, e-mail: common-commits-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-commits-help@hadoop.apache.org