This is an automated email from the ASF dual-hosted git repository.
junrao 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 b11b2cdd6a3 MINOR: Followup KIP-1161 upgrade and test (#20989)
b11b2cdd6a3 is described below
commit b11b2cdd6a30ae56a0c806b4b279e3b78721a260
Author: Ken Huang <[email protected]>
AuthorDate: Wed Nov 26 03:50:37 2025 +0800
MINOR: Followup KIP-1161 upgrade and test (#20989)
FYI: https://github.com/apache/kafka/pull/20960#discussion_r2557144873
Reviewers: Jun Rao <[email protected]>
---
.../src/test/java/org/apache/kafka/common/config/ConfigDefTest.java | 4 ++--
docs/upgrade.html | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git
a/clients/src/test/java/org/apache/kafka/common/config/ConfigDefTest.java
b/clients/src/test/java/org/apache/kafka/common/config/ConfigDefTest.java
index a9ef4690a82..c4200e82b0f 100644
--- a/clients/src/test/java/org/apache/kafka/common/config/ConfigDefTest.java
+++ b/clients/src/test/java/org/apache/kafka/common/config/ConfigDefTest.java
@@ -803,8 +803,8 @@ public class ConfigDefTest {
assertEquals("Configuration 'test.config' values must not be null.",
exception12.getMessage());
ConfigException exception13 = assertThrows(ConfigException.class, ()
-> allowAnyNonDuplicateValues.ensureValid("test.config", List.of()));
assertEquals("Configuration 'test.config' must not be empty. Valid
values include: any non-empty value", exception13.getMessage());
- ConfigException exception14 = assertThrows(ConfigException.class, ()
-> allowAnyNonDuplicateValues.ensureValid("test.config", List.of("a", "",
"b")));
- assertEquals("Configuration 'test.config' values must not be empty.",
exception14.getMessage());
+ ConfigException exception14 = assertThrows(ConfigException.class, ()
-> allowAnyNonDuplicateValues.ensureValid("test.config", List.of("a", "a")));
+ assertEquals("Configuration 'test.config' values must not be
duplicated.", exception14.getMessage());
ConfigException exception15 = assertThrows(ConfigException.class, ()
-> allowAnyNonDuplicateValues.ensureValid("test.config", List.of("")));
assertEquals("Configuration 'test.config' values must not be empty.",
exception15.getMessage());
ConfigException exception16 = assertThrows(ConfigException.class, ()
-> allowAnyNonDuplicateValues.ensureValid("test.config", List.of("a", "",
"b")));
diff --git a/docs/upgrade.html b/docs/upgrade.html
index 1796b974d9d..fe1b8b80d49 100644
--- a/docs/upgrade.html
+++ b/docs/upgrade.html
@@ -143,7 +143,7 @@
<ul>
<li>Null values are no longer accepted for most LIST-type
configurations. Exceptions apply only to
configurations that have null as their default value,
as users cannot explicitly assign null values
- in configuration files.</li>
+ in configuration files or through the API.</li>
<li>Most LIST-type configurations no longer accept
duplicate entries, except in cases where duplicates
are explicitly supported. For backward compatibility,
if users configure duplicate entries when they
are not accepted, duplicate entries will be ignored
and a warning will be logged.</li>
