Jackie-Jiang opened a new pull request, #15405: URL: https://github.com/apache/pinot/pull/15405
#15404 moves the consumer lock acquisition into the consumer thread, and it is no longer blocking the `OFFLINE -> CONSUMING` state transition. Because of this, the `CONSUMING -> ONLINE` state transition can be processed before consumer lock is acquired. This is in general desired, but the corner case described in #15399 won't be handled. It should be an extremely rare corner case, and is very hard to handle without introducing significant overhead, thus added a TODO to revisit if necessary. This PR simplifies the logic in `ConsumerCoordinator`, and also fixes a bug in `RealtimeTableDataManager` for uploaded segment. -- 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
