This is an automated email from the ASF dual-hosted git repository.

chia7712 pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/kafka.git


The following commit(s) were added to refs/heads/trunk by this push:
     new 74484d223ad MINOR: Fixing unchecked warning in partition strategy test 
(#18541)
74484d223ad is described below

commit 74484d223ad8c70e2ee01b5c0fb6f378e47751ed
Author: Apoorv Mittal <apoorvmitta...@gmail.com>
AuthorDate: Wed Jan 15 20:33:57 2025 +0000

    MINOR: Fixing unchecked warning in partition strategy test (#18541)
    
    Reviewers: Viktor Somogyi-Vass <viktorsomo...@gmail.com>, Chia-Ping Tsai 
<chia7...@gmail.com>
---
 .../apache/kafka/server/share/fetch/PartitionMaxBytesStrategyTest.java | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git 
a/share/src/test/java/org/apache/kafka/server/share/fetch/PartitionMaxBytesStrategyTest.java
 
b/share/src/test/java/org/apache/kafka/server/share/fetch/PartitionMaxBytesStrategyTest.java
index 3c6c7ad2207..07338673832 100644
--- 
a/share/src/test/java/org/apache/kafka/server/share/fetch/PartitionMaxBytesStrategyTest.java
+++ 
b/share/src/test/java/org/apache/kafka/server/share/fetch/PartitionMaxBytesStrategyTest.java
@@ -23,7 +23,6 @@ import 
org.apache.kafka.server.share.fetch.PartitionMaxBytesStrategy.StrategyTyp
 
 import org.junit.jupiter.api.Test;
 
-import java.util.Collections;
 import java.util.LinkedHashMap;
 import java.util.LinkedHashSet;
 import java.util.List;
@@ -56,7 +55,7 @@ public class PartitionMaxBytesStrategyTest {
             100, partitions, 0));
         // empty partitions set.
         assertThrows(IllegalArgumentException.class, () -> 
PartitionMaxBytesStrategy.checkValidArguments(
-            100, Collections.EMPTY_SET, 20));
+            100, Set.of(), 20));
         // partitions is null.
         assertThrows(IllegalArgumentException.class, () -> 
PartitionMaxBytesStrategy.checkValidArguments(
             100, null, 20));

Reply via email to