You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by "navina (via GitHub)" <gi...@apache.org> on 2023/02/23 18:55:32 UTC

[GitHub] [pinot] navina commented on pull request #10310: Consumer lag unknown

navina commented on PR #10310:
URL: https://github.com/apache/pinot/pull/10310#issuecomment-1442275988

   > I think the issue with getUpstreamLatestOffset being null happens at around the time that we're changing the consuming segment/sealing it. 
   
   @mneedham : I don't think the segment commit should interfere with consumer state. any chance you saw other logs on the server around the time the `null` response was returned? 
   
   >I saw you annotate partitionToOffsetMap as deprecated in https://github.com/apache/pinot/pull/9515, is there a plan to change the api?
   
   @Jackie-Jiang : Yes. I think it should be ok to change the API now by removing the `partitionToOffsetMap` since the deprecation is already past 1 release. 
   


-- 
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-unsubscribe@pinot.apache.org

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


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