rohitrmd commented on a change in pull request #9744:
URL: https://github.com/apache/kafka/pull/9744#discussion_r567356968
##########
File path:
streams/src/main/java/org/apache/kafka/streams/processor/internals/AbstractProcessorContext.java
##########
@@ -45,7 +45,6 @@
private boolean initialized;
protected ProcessorRecordContext recordContext;
protected ProcessorNode<?, ?, ?, ?> currentNode;
- private long currentSystemTimeMs;
Review comment:
@mjsax also I am confused about what time do we want to return when
currentSystemTimeMs() is called from the ProcessorContext hierarchy. If it's
called from AbstractProcessorContext object reference, should we return
cachedSystemTimeMs, and if it's called from ProcessorContextImpl object
reference, should we return streamTask.currentSystemTimeMs()? Or we want
completely new method to return the cached time when
ProcessorContextImpl#forwardInternal() is called?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
