ankitsultana opened a new pull request, #15562: URL: https://github.com/apache/pinot/pull/15562
This PR contains minor changes to some of the MSE code that is used by the existing optimizer / queries. These changes are required for the Physical Optimizer changes I am working on. ### Summary #### Add ExchangeStrategy to ExchangeNode This is required because Physical Optimizer uses the Exchange Strategy enum to infer the type of exchange. This value will be used in the new fragmenter/mailbox-assignment I have implemented (which will be isolated from the current query code-path). #### Remove Logical Node Refs in SetOpNode This is because we will use Physical* nodes #### WorkerManager Getters I don't like this change but it's temporary to avoid creating a ruckus in the already chaotic QueryEnvironment. #### SingletonExchange Precondition InMemory mailbox is not necessary for SingletonExchange, since Singleton simply means 1:1 worker mapping and doesn't mean "local". -- 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
