[KAFKA-13899] Inconsistent error codes returned from AlterConfig APIs

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.2.1
• Assignee: Jason Gustafson
• Reporter: Jason Gustafson
• Created: 13/May/22 16:40
• Updated: 17/May/22 00:44
• Resolved: 17/May/22 00:42

Description

In the AlterConfigs/IncrementalAlterConfigs zk handler, we return INVALID_REQUEST and INVALID_CONFIG inconsistently. The problem is in `LogConfig.validate`. We may either return `ConfigException` or `InvalidConfigException`. When the first of these is thrown, we catch it and convert to INVALID_REQUEST. It seems more consistent to convert to INVALID_CONFIG.

Note that the kraft implementation returns INVALID_CONFIG consistently.

Issue Links

• links to: GitHub Pull Request #12162