abhinav

Yes I will look into this.

@vvivekiyer : Just to understand the design here, are you suggesting we move https://github.com/apache/pinot/blob/67344859c270b9e64638b6691d456e57a7570403/pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java#L671 inside a standalone method called `getClientIp` in `org.apache.pinot.broker.api.HttpRequesterIdentity` ?

Another option is we could also do below to avoid the issue and explicit check `if (enableClientIpLogging && requesterIdentity instanceof HttpRequesterIdentity) ` instead of https://github.com/apache/pinot/blob/67344859c270b9e64638b6691d456e57a7570403/pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java#L678