[spark] branch master updated: [SPARK-45576][CORE] Remove unnecessary debug logs in ReloadingX509TrustManagerSuite

[mridulm80]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 4901548d4c3 [SPARK-45576][CORE] Remove unnecessary debug logs in 
ReloadingX509TrustManagerSuite
4901548d4c3 is described below

commit 4901548d4c36ac5988bcc04501057de40712e66d
Author: Hasnain Lakhani <hasnain.lakh...@databricks.com>
AuthorDate: Tue Oct 17 23:48:19 2023 -0500

    [SPARK-45576][CORE] Remove unnecessary debug logs in 
ReloadingX509TrustManagerSuite
    
    ### What changes were proposed in this pull request?
    
    Remove debug logs that were left in by accident.
    
    ### Why are the changes needed?
    
    These were not intended to be committed
    
    ### Does this PR introduce _any_ user-facing change?
    
    No
    
    ### How was this patch tested?
    
    CI
    
    ### Was this patch authored or co-authored using generative AI tooling?
    
    No
    
    Closes #43404 from hasnain-db/remove-logs.
    
    Authored-by: Hasnain Lakhani <hasnain.lakh...@databricks.com>
    Signed-off-by: Mridul Muralidharan <mridul<at>gmail.com>
---
 .../apache/spark/network/ssl/ReloadingX509TrustManagerSuite.java    | 6 ------
 1 file changed, 6 deletions(-)

diff --git 
a/common/network-common/src/test/java/org/apache/spark/network/ssl/ReloadingX509TrustManagerSuite.java
 
b/common/network-common/src/test/java/org/apache/spark/network/ssl/ReloadingX509TrustManagerSuite.java
index 7e2cc38e70b..dbd71f987d4 100644
--- 
a/common/network-common/src/test/java/org/apache/spark/network/ssl/ReloadingX509TrustManagerSuite.java
+++ 
b/common/network-common/src/test/java/org/apache/spark/network/ssl/ReloadingX509TrustManagerSuite.java
@@ -37,8 +37,6 @@ import static org.apache.spark.network.ssl.SslSampleConfigs.*;
 
 public class ReloadingX509TrustManagerSuite {
 
-  private final Logger logger = 
LoggerFactory.getLogger(ReloadingX509TrustManagerSuite.class);
-
   /**
    * Waits until reload count hits the requested value, sleeping 100ms at a 
time.
    * If the maximum number of attempts is hit, throws a RuntimeException
@@ -280,8 +278,6 @@ public class ReloadingX509TrustManagerSuite {
             new ReloadingX509TrustManager("jks", trustStoreSymlink, 
"password", 1);
     assertEquals(1, tm.getReloadInterval());
     assertEquals(0, tm.reloadCount);
-    logger.info("TRUST STORE 1 IS" + trustStore1);
-    logger.info("TRUST STORE 2 IS " + trustStore2);
     try {
       tm.init();
       assertEquals(1, tm.getAcceptedIssuers().length);
@@ -289,10 +285,8 @@ public class ReloadingX509TrustManagerSuite {
       assertEquals(0, tm.reloadCount);
 
       // Repoint to trustStore2, which has another cert
-      logger.info("REPOINTING SYMLINK!!!");
       trustStoreSymlink.delete();
       Files.createSymbolicLink(trustStoreSymlink.toPath(), 
trustStore2.toPath());
-      logger.info("REPOINTED!!!");
 
       // Wait up to 5s until we reload
       waitForReloadCount(tm, 1, 50);


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org