You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@kafka.apache.org by "jingjia88 (via GitHub)" <gi...@apache.org> on 2025/05/19 19:10:56 UTC
[PR] Kafka-19314: Remove unnecessary code of closing snapshotWriter [kafka]
jingjia88 opened a new pull request, #19763:
URL: https://github.com/apache/kafka/pull/19763
- Remove redundant close of `snapshotWriter`.
- `snapshotWriter` is already closed by `RaftSnapshotWriter#writer`.
--
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
Re: [PR] KAFKA-19314: Remove unnecessary code of closing snapshotWriter [kafka]
Posted by "chia7712 (via GitHub)" <gi...@apache.org>.
chia7712 merged PR #19763:
URL: https://github.com/apache/kafka/pull/19763
--
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
Re: [PR] Kafka-19314: Remove unnecessary code of closing snapshotWriter [kafka]
Posted by "Yunyung (via GitHub)" <gi...@apache.org>.
Yunyung commented on PR #19763:
URL: https://github.com/apache/kafka/pull/19763#issuecomment-2893402985
Looks good. Please update the PR title, Kafka -> KAFKA.
--
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
Re: [PR] KAFKA-19314: Remove unnecessary code of closing snapshotWriter [kafka]
Posted by "jingjia88 (via GitHub)" <gi...@apache.org>.
jingjia88 commented on PR #19763:
URL: https://github.com/apache/kafka/pull/19763#issuecomment-2893558162
> Looks good. Please update the PR title, Kafka -> KAFKA.
Thanks, updated the PR title as suggested. PTAL.
--
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