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 <[email protected]>
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 <[email protected]>
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: [email protected]
For additional commands, e-mail: [email protected]
