apoorvmittal10 commented on PR #17283: URL: https://github.com/apache/kafka/pull/17283#issuecomment-2379150535
> Could you please clarify why you decided not to use `CompletableFuture` ([#16274 (comment)](https://github.com/apache/kafka/pull/16274#discussion_r1718405279))? Hi @chia7712, though my inclination was towards keeping the code with CompletableFuture as in future if we think about persisting state when records are acquired then it will give us flexibility. But given the code with CompletableFuture adds a lot more complexity in handling scenarios, as again pointed out during introducing purgatory, https://github.com/apache/kafka/pull/17177#discussion_r1758382532. Hence it's better to remove it for now. Once we have concrete use case then we should think of adding it back. -- 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
