Skip to content

Commit a62f0ac

Browse files
Fixing PR comments
1 parent ea7c9b7 commit a62f0ac

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

client/src/main/java/io/split/telemetry/synchronizer/TelemetrySyncTask.java

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,9 @@
22

33
import com.google.common.annotations.VisibleForTesting;
44
import com.google.common.util.concurrent.ThreadFactoryBuilder;
5+
import io.split.client.SplitManagerImpl;
6+
import org.slf4j.Logger;
7+
import org.slf4j.LoggerFactory;
58

69
import java.util.concurrent.Executors;
710
import java.util.concurrent.ScheduledExecutorService;
@@ -10,6 +13,7 @@
1013

1114
public class TelemetrySyncTask {
1215

16+
private static final Logger _log = LoggerFactory.getLogger(TelemetrySyncTask.class);
1317
private final ScheduledExecutorService _telemetrySyncScheduledExecutorService;
1418
private final TelemetrySynchronizer _telemetrySynchronizer;
1519
private final int _telemetryRefreshRate;
@@ -44,7 +48,7 @@ public void stopScheduledTask(long splitCount, long segmentCount, long segmentKe
4448
try {
4549
_telemetrySynchronizer.finalSynchronization(splitCount, segmentCount, segmentKeyCount);
4650
} catch (Exception e) {
47-
e.printStackTrace();
51+
_log.warn("Error trying to send telemetry stats.");
4852
}
4953
_telemetrySyncScheduledExecutorService.shutdown();
5054
}

0 commit comments

Comments
 (0)