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-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
