[GitHub] [pinot] jugomezv commented on a diff in pull request #10121: Add a tracker for end-to-end consumption delay of events.

[via GitHub]

jugomezv commented on code in PR #10121:
URL: https://github.com/apache/pinot/pull/10121#discussion_r1083151558


##########
pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/LLRealtimeSegmentDataManager.java:
##########
@@ -615,7 +615,8 @@ private boolean processStreamEvents(MessageBatch 
messagesAndOffsets, long idlePi
       }
     } else if (!prematureExit) {
       // Record Pinot ingestion delay as zero since we are up-to-date and no 
new events
-      
_realtimeTableDataManager.updateIngestionDelay(System.currentTimeMillis(), 
_partitionGroupId);
+      
_realtimeTableDataManager.updateIngestionDelay(System.currentTimeMillis(), 
System.currentTimeMillis(),

Review Comment:
   Addressed in commit 9bb0befe9a55c01bb3a5066a4b30137469819f38



-- 
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: commits-unsubscr...@pinot.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org