MINOR: Remove exitMessage.set() call in TopicBasedRemoteLogMetadataManagerTest (#20563)

- **Reasons:** In this case, the `exit(int statusCode)` method invokes
`exit(statusCode, null)`, which means
the `message` argument is always `null` in this code path. As a result,
assigning
`exitMessage` has no effect and can be safely removed.

- **Changes:** Remove a redundant field assignment.

Reviewers: Chia-Ping Tsai <chia7712@gmail.com>
This commit is contained in:
Chang-Chi Hsu 2025-09-20 12:04:10 +02:00 committed by GitHub
parent c49ab6b4ae
commit 5919762009
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 0 additions and 3 deletions

View File

@ -46,7 +46,6 @@ import java.util.concurrent.ExecutionException;
import java.util.concurrent.TimeUnit;
import java.util.concurrent.atomic.AtomicBoolean;
import java.util.concurrent.atomic.AtomicInteger;
import java.util.concurrent.atomic.AtomicReference;
import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.junit.jupiter.api.Assertions.assertFalse;
@ -344,13 +343,11 @@ public class TopicBasedRemoteLogMetadataManagerTest {
// Set up a custom exit procedure for testing
final AtomicBoolean exitCalled = new AtomicBoolean(false);
final AtomicInteger exitCode = new AtomicInteger(-1);
final AtomicReference<String> exitMessage = new AtomicReference<>();
// Set custom exit procedure that won't actually exit the process
Exit.setExitProcedure((statusCode, message) -> {
exitCalled.set(true);
exitCode.set(statusCode);
exitMessage.set(message);
});
try (TopicBasedRemoteLogMetadataManager rlmm = new TopicBasedRemoteLogMetadataManager()) {