You are viewing a plain text version of this content. The canonical link for it is here.

Posted to jira@kafka.apache.org by "ijuma (via GitHub)" <gi...@apache.org> on 2023/03/04 19:28:34 UTC

[GitHub] [kafka] ijuma commented on pull request #13309: MINOR Moved a few log segment util methods from LocalLog to LogFileUtils

ijuma commented on PR #13309:
URL: https://github.com/apache/kafka/pull/13309#issuecomment-1454850030

   I didn't see this until now. I actually think the right approach is to push some of these methods to the respective classes (eg if it's related to an index, the method should be in the index class). I starting working on a PR for that and will submit it later.


-- 
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-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org