[ 
https://issues.apache.org/jira/browse/KAFKA-10871?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Rohit Deshpande resolved KAFKA-10871.
-------------------------------------
    Resolution: Not A Bug

> StreamTask shouldn't take WallClockTime as input parameter in process method
> ----------------------------------------------------------------------------
>
>                 Key: KAFKA-10871
>                 URL: https://issues.apache.org/jira/browse/KAFKA-10871
>             Project: Kafka
>          Issue Type: Bug
>          Components: streams
>    Affects Versions: 2.7.0
>            Reporter: Rohit Deshpande
>            Assignee: Rohit Deshpande
>            Priority: Major
>              Labels: newbie
>
> While working on https://issues.apache.org/jira/browse/KAFKA-10062 I realized 
> process method in StreamTask is taking
> wallClockTime as input parameter which is redundant as StreamTask already 
> contains 
> time(https://github.com/apache/kafka/blob/2.5.0/streams/src/main/java/org/apache/kafka/streams/processor/internals/StreamTask.java#L75)
>  field which represents wallClockTime.
> In process method 
> (https://github.com/apache/kafka/blob/2.7/streams/src/main/java/org/apache/kafka/streams/processor/internals/StreamTask.java#L664),
>  wallClockTime can be passed from StreamTask's time field itself. 
> This method was changed as part of pr: 
> https://github.com/apache/kafka/pull/7997. 
> As part of https://issues.apache.org/jira/browse/KAFKA-10062, I believe 
> wallClockTime need not be stored in 
> ProcessorContext(https://github.com/apache/kafka/blob/trunk/streams/src/main/java/org/apache/kafka/streams/processor/internals/AbstractProcessorContext.java#L48)
>  but should be fetched from StreamTask's time field. Reference pr: 
> https://github.com/apache/kafka/pull/9744



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to