d8tltanc edited a comment on pull request #8610:
URL: https://github.com/apache/kafka/pull/8610#issuecomment-624421790


   I think `ClientQuotaEntity` is only used in client quota alternation. But 
the description doesn't use that.
   Update: I've pushed my latest local changes if that helps. My patch is 
assuming the server is treating `match` value null as the default. The original 
design might be assuming `match` value Optional.empty() as the default and 
there might be some inconsistencies. The failed tests are 
**testDescribeClientQuotasMatchExact** and 
**testDescribeClientQuotasMatchPartial**


----------------------------------------------------------------
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.

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


Reply via email to