gortiz opened a new pull request, #11205: URL: https://github.com/apache/pinot/pull/11205
This PR includes a deep change in V2 scheduler in order to do not use NOOP blocks. The NOOP implementation used to create a coroutine system in V2 has been proven to be very error prone and even if it is possible to implent the same idea in a safer way, we think it is better to use a more imperative and easy to understand scheduler. The main disadvantage is that we need to have one thread per active OpChain, but don't expect to have thousand of running OpChains, so it shouldn't be a problem. In short term, we may be able to use Loom, which would make the code even easier. Even if Git thinks I'm the author of these changes, most of the ideas and even lines were originally created by @walterddr in its own fork. Given that it was very difficult to rebase these changes, I've manually applied these changes to a more recent master branch. -- 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
