chia7712 commented on code in PR #9393:
URL: https://github.com/apache/kafka/pull/9393#discussion_r2671258699
##########
clients/src/main/java/org/apache/kafka/clients/admin/Admin.java:
##########
@@ -1367,10 +1378,9 @@ AlterUserScramCredentialsResult
alterUserScramCredentials(List<UserScramCredenti
* <p>
* This operation is supported by brokers with version 2.7.0 or higher.
- * @param featureUpdates the map of finalized feature name to {@link
FeatureUpdate}
- * @param options the options to use
- *
- * @return the {@link UpdateFeaturesResult} containing the
result
+ * @param featureUpdates the map of finalized feature name to {@link
FeatureUpdate}
+ * @param options the options to use
+ * @return the {@link UpdateFeaturesResult} containing the result
*/
UpdateFeaturesResult updateFeatures(Map<String, FeatureUpdate>
featureUpdates, UpdateFeaturesOptions options);
Review Comment:
It appears we forgot to add an overload for updateFeatures. @kowshik @junrao
WDYT? Is it worth adding consistency for now?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
