[
https://issues.apache.org/jira/browse/KAFKA-17131?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17906648#comment-17906648
]
Matthias J. Sax edited comment on KAFKA-17131 at 12/18/24 8:33 AM:
-------------------------------------------------------------------
Did already start working on this and related ticket 13772. I expect both to be
tricky tasks, with quite some refactoring... Also Sophie did recently assign
17178 to herself – as long as `ProcessorContext` is in use (by not yet
deprecated methods), we cannot deprecate the interface, so 17178 blocks this
ticket to get finalized. I am currently working on internal cleanup to
"prepare" this ticket to get finished (but I am not sure if I can finish this
ticket).
So after I am done with my PR, we can revisit and you might be able to pick it
up, assuming 17178 is done.
Given that `ProcessorContext` was already deprecated via KIP-478, I actually
believe we don't need a KIP (and if in doubt, we should just piggy-pack it on
17178)
was (Author: mjsax):
Did already start working on this and related ticket 13772. I expect both to be
tricky tasks, with quite some refactoring... Also Sophie did recently assign
17178 to herself – as long as `ProcessorContext` is in use (by not yet
deprecated methods), we cannot deprecate the interface, so 17178 blocks this
ticket to get finalized. I am currently working on internal cleanup to
"prepare" this ticket to get finished (but I am not sure if I can finish this
ticket).
So after I am done with my PR, we can revisit and you might be able to pick it
up, assuming 17178 is done.
> Cleanup `ProcessorContext` usage
> --------------------------------
>
> Key: KAFKA-17131
> URL: https://issues.apache.org/jira/browse/KAFKA-17131
> Project: Kafka
> Issue Type: Improvement
> Components: streams
> Reporter: Matthias J. Sax
> Assignee: Matthias J. Sax
> Priority: Major
> Labels: needs-kip
>
> We did replace `Processor` and related classes with `api.Processor` et al. –
> However, there is also `ProcessorContext` which was deprecated via KIP-478
> (AK 2.7) but not marked as deprecated as it was in use in many other places.
> Many of the methods which used the old `ProcessorContext` got removed in AK
> 4.0 release, and some others are already deprecated.
> However, `ProcessorContext` is still in use in internal code, and we should
> try to refactor internal code (especially for state stores) to move off the
> old interface, so we can eventually remove it (most likely in AK 5.0).
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
