[multistage][cleanup] clean up opChain wake up call API

[walterddr]

Summary

OpChainSchedulerService now accepts OpChain ID for onDataAvailable

• decouple Mailbox concept from OpChain
• MailboxService use callback method that accepts directly OpChainID instead of MailboxID

TODO

• we should decouple MailboxService callback into --> mailboxID --> opChainID lookup method and onDataAvailable callback method
• allow non-mailbox based wake up of OpChain for example local subquery result using pipeline breaker operator

[codecov-commenter]

Codecov Report

Merging #10710 (4249c61) into master (b72051c) will decrease coverage by 1.41%.
The diff coverage is 85.71%.

@@             Coverage Diff              @@
##             master   #10710      +/-   ##
============================================
- Coverage     70.31%   68.91%   -1.41%     
- Complexity     5641     6428     +787     
============================================
  Files          2112     2112              
  Lines        114048   114047       -1     
  Branches      17223    17222       -1     
============================================
- Hits          80196    78590    -1606     
- Misses        28255    29942    +1687     
+ Partials       5597     5515      -82     

Flag	Coverage Δ
integration1	?
integration2	24.05% <0.00%> (+0.06%)	⬆️
unittests1	67.82% <85.71%> (+<0.01%)	⬆️
unittests2	13.81% <0.00%> (-0.02%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...ot/query/runtime/executor/RoundRobinScheduler.java	86.58% <50.00%> (-2.58%)	⬇️
...org/apache/pinot/query/mailbox/MailboxService.java	77.14% <100.00%> (ø)
...g/apache/pinot/query/mailbox/ReceivingMailbox.java	73.80% <100.00%> (ø)
...uery/runtime/executor/OpChainSchedulerService.java	95.16% <100.00%> (ø)

... and 152 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more