You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@kafka.apache.org by "Kirk True (Jira)" <ji...@apache.org> on 2021/07/06 18:37:00 UTC
[jira] [Assigned] (KAFKA-12989) MockClient should respect the
request matcher passed to prepareUnsupportedVersionResponse
[ https://issues.apache.org/jira/browse/KAFKA-12989?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Kirk True reassigned KAFKA-12989:
---------------------------------
Assignee: Kirk True
> MockClient should respect the request matcher passed to prepareUnsupportedVersionResponse
> -----------------------------------------------------------------------------------------
>
> Key: KAFKA-12989
> URL: https://issues.apache.org/jira/browse/KAFKA-12989
> Project: Kafka
> Issue Type: Bug
> Reporter: David Jacot
> Assignee: Kirk True
> Priority: Major
>
> The {{MockClient}} offers the possibility to prepare an unsupported version response with {{MockClient#prepareUnsupportedVersionResponse}}. That method accepts a {{RequestMatcher}} but it is never applied. It should be or we should remove the matcher from the method.
> {code:java}
> UnsupportedVersionException unsupportedVersionException = null;
> if (futureResp.isUnsupportedRequest) {
> unsupportedVersionException = new UnsupportedVersionException(
> "Api " + request.apiKey() + " with version " + version);
> } else {
> AbstractRequest abstractRequest = request.requestBuilder().build(version);
> if (!futureResp.requestMatcher.matches(abstractRequest))
> throw new IllegalStateException("Request matcher did not match next-in-line request "
> + abstractRequest + " with prepared response " + futureResp.responseBody);
> }
> ClientResponse resp = new ClientResponse(request.makeHeader(version), request.callback(), request.destination(),
> request.createdTimeMs(), time.milliseconds(), futureResp.disconnected,
> unsupportedVersionException, null, futureResp.responseBody);
> {code}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)