You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@kafka.apache.org by "Owen C.H. Leung (Jira)" <ji...@apache.org> on 2023/06/15 02:45:00 UTC
[jira] [Commented] (KAFKA-15085) Make Timer.java implement AutoCloseable
[ https://issues.apache.org/jira/browse/KAFKA-15085?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17732824#comment-17732824 ]
Owen C.H. Leung commented on KAFKA-15085:
-----------------------------------------
Hi [~divijvaidya] , can you assign this issue to me ? I'm eager to pick this up.
Also, can I ask if you can help to review my PR which aims to solve KAFKA-14712 ?
https://github.com/apache/kafka/pull/13773
> Make Timer.java implement AutoCloseable
> ---------------------------------------
>
> Key: KAFKA-15085
> URL: https://issues.apache.org/jira/browse/KAFKA-15085
> Project: Kafka
> Issue Type: Improvement
> Components: core
> Reporter: Divij Vaidya
> Priority: Minor
> Labels: Newbie, newbie
>
> Many automatic bug finders will flag a warning if an object of class which is marked as AutoCloseable but is not closed properly in the code. Hence, as a best practice we should implement AutoCloseable for classes which require resources to be released after shutdown.
> Timer.java should implement AutoCloseable and ShutDown should be replaced with close() method.
> Note that this interface change does not require a KIP since Timer.java is not a user facing public class.
> This was discussed here: [https://github.com/apache/kafka/pull/13820#discussion_r1222654614]
--
This message was sent by Atlassian Jira
(v8.20.10#820010)