RivenSun2 commented on pull request #11800:
URL: https://github.com/apache/kafka/pull/11800#issuecomment-1050676944


   @showuon 
   Sorry, I made a mistake, the `map` returned by 
`config.valuesWithPrefixOverride(prefix)` is not a normal map, it is of type 
`AbstractConfig.RecordingMap`.
   `RecordingMap#get(Object key)` will call the `AbstractConfig#ignore(String 
key)` method to change the `AbstractConfig.used` variable.
   Forgive my mistakes.
   
   However, after `valuesWithPrefixOverride.get("sasl.mechanism")` add 
`assertFalse(config.unknown().contains("prefix.sasl.mechanism"));`,
   testCase will fail to verify; because unknownKeys is only affected by 
originals and values, the value is originalKeys.removeAll(valueKeys).


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to