fix: removing a (maybe unnecessary) upstream request on group emission

[ozangunalp]

each emitted item already requests upstream items

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.16%. Comparing base (a1998df) to head (baef9fb).
⚠️ Report is 31 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff              @@
##               main    #2032      +/-   ##
============================================
- Coverage     89.18%   89.16%   -0.03%     
- Complexity     3120     3121       +1     
============================================
  Files           412      412              
  Lines         13295    13294       -1     
  Branches       1688     1688              
============================================
- Hits          11857    11853       -4     
- Misses          812      813       +1     
- Partials        626      628       +2     

Files with missing lines	Coverage Δ
...mallrye/mutiny/operators/multi/MultiGroupByOp.java	86.08% <ø> (-0.07%)	⬇️

... and 13 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[jponge]

It makes sense to me, I'd just add a test to make it obvious though 😃

[jponge]

@ozangunalp Do you see how a test could be added? Otherwise LGTM

[ozangunalp]

It makes sense to me, I'd just add a test to make it obvious though 😃

I think I did but forgot to push it :D let me look at it.

[ozangunalp]

I had modified the testUpstreamRequestsNotBlownOutOfProportion test to check for the exact number of requests.