Skip to content

Commit b1d36dd

Browse files
committed
refactor: check log level before logging
1 parent 71902e5 commit b1d36dd

File tree

2 files changed

+9
-0
lines changed

2 files changed

+9
-0
lines changed

wrapper/src/main/java/com/amazon/awslabs/jdbc/hostlistprovider/AuroraHostListProvider.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -540,6 +540,10 @@ private void validateHostPatternSetting(String hostPattern) {
540540
}
541541

542542
private void logTopology(final List<HostSpec> topology) {
543+
if (!LOGGER.isLoggable(Level.FINER)) {
544+
return;
545+
}
546+
543547
StringBuilder msg = new StringBuilder();
544548
for (int i = 0; i < topology.size(); i++) {
545549
HostSpec hostInfo = topology.get(i);

wrapper/src/main/java/com/amazon/awslabs/jdbc/plugin/failover/ClusterAwareWriterFailoverHandler.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,7 @@
3535
import java.util.concurrent.Executors;
3636
import java.util.concurrent.Future;
3737
import java.util.concurrent.TimeUnit;
38+
import java.util.logging.Level;
3839
import java.util.logging.Logger;
3940

4041
/**
@@ -448,6 +449,10 @@ private void performFinalCleanup() {
448449
}
449450

450451
private void logTopology() {
452+
if (!LOGGER.isLoggable(Level.FINER)) {
453+
return;
454+
}
455+
451456
StringBuilder msg = new StringBuilder();
452457
for (int i = 0; i < this.currentTopology.size(); i++) {
453458
HostSpec hostInfo = this.currentTopology.get(i);

0 commit comments

Comments
 (0)