ashmeet13 commented on PR #12414: URL: https://github.com/apache/kafka/pull/12414#issuecomment-1234551652
Thank you @jsancio for the review. Have made the changes. For the following comment - >It looks like this method is only used by tests. I think it is okay for this to have an "unknown reason" for now. Currently the function in discussion has no logging taking place hence I haven't added the `UnkownReason` in. Should I update the function to log the same? -- 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: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
