Skip to content

Commit 8d701fb

Browse files
committed
Fix null threadFactory in EventsTask when not configured
EventsTask now handles null threadFactory by using the default Executors.newSingleThreadScheduledExecutor() without a factory argument. This allows tests to run without configuring a thread factory in the SDK config. Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> AI-Session-Id: c83b3557-2c02-4d13-aaa4-273b7340163d AI-Tool: claude-code AI-Model: unknown
1 parent 1881107 commit 8d701fb

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

tracker/src/main/java/io/split/client/events/EventsTask.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,9 @@ public EventsTask(EventsStorageConsumer eventsStorageConsumer,
3838
_eventsStorageConsumer = Objects.requireNonNull(eventsStorageConsumer);
3939
_sendIntervalMillis = sendIntervalMillis;
4040
_eventsSender = Objects.requireNonNull(eventsSender);
41-
_senderScheduledExecutorService = Executors.newSingleThreadScheduledExecutor(threadFactory);
41+
_senderScheduledExecutorService = threadFactory != null
42+
? Executors.newSingleThreadScheduledExecutor(threadFactory)
43+
: Executors.newSingleThreadScheduledExecutor();
4244
}
4345

4446
public void start(){

0 commit comments

Comments
 (0)