ankitflow001 commented on PR #24076: URL: https://github.com/apache/kafka/pull/24076#issuecomment-6684011649
> @ankitflow001 thanks for the PR. I think there are more classes in 'clients' module like ClusterConnectionStates, AbstractMembershipManager.java, AbstractStickyAssignor.java, etc ... which lack `@Override` > > May be they can be part of another PR too. > @ankitflow001 thanks for the PR. I think there are more classes in 'clients' module like ClusterConnectionStates, AbstractMembershipManager.java, AbstractStickyAssignor.java, etc ... which lack `@Override` > > May be they can be part of another PR too. Thanks @muralibasani! I'll prepare a follow-up PR covering the other clients classes missing @Override — ClusterConnectionStates, AbstractMembershipManager, AbstractStickyAssignor, and any others I find. For this one, could you approve the pending CI workflows when you get a chance? As a first-time contributor, the remaining 4 workflows need maintainer approval to run. -- 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]
