Skip to content

Commit

Permalink
[KAFKA-15137] Do not log entire request payload in KRaftControllerCha…
Browse files Browse the repository at this point in the history
…nnelManager (apache#13988)

Reviewers: David Arthur <[email protected]>
  • Loading branch information
ahuang98 authored Jul 11, 2023
1 parent 6afcfba commit 5b5f6fc
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,18 @@ protected Builder(ApiKeys api, short version, int controllerId, int controllerEp
this.brokerEpoch = brokerEpoch;
this.kraftController = kraftController;
}

public int controllerId() {
return controllerId;
}

public int controllerEpoch() {
return controllerEpoch;
}

public long brokerEpoch() {
return brokerEpoch;
}
}

protected AbstractControlRequest(ApiKeys api, short version) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,9 @@ class ControllerChannelManager(controllerEpoch: () => Int,
case Some(stateInfo) =>
stateInfo.messageQueue.put(QueueItem(request.apiKey, request, callback, time.milliseconds()))
case None =>
warn(s"Not sending request $request to broker $brokerId, since it is offline.")
warn(s"Not sending request ${request.apiKey.name} with controllerId=${request.controllerId()}, " +
s"controllerEpoch=${request.controllerEpoch()}, brokerEpoch=${request.brokerEpoch()} " +
s"to broker $brokerId, since it is offline.")
}
}
}
Expand Down

0 comments on commit 5b5f6fc

Please sign in to comment.