supabase / supavisor / 29432428714

78%

Build Type

push

github

Committed by web-flow

Commit Message

fix: wait for all DB replies before releasing backend (#1079)

## What kind of change does this PR introduce?

Avoids prematurely releasing the backend on the first-found
`ReadyForQuery` message.

## What is the current behavior?

The client hangs waiting for the rest of the replies, but Supavisor
already releases the backend early, causing the remaining replies to be
lost.

## What is the new behavior?

The client handler now communicates the expected `ReadyForQuery`
messages to wait for, so we only release the backend when we are done
with the batch.

## Additional context

Resolves #1061 

I spent good time on the tests. The integration tests correctly catch
the regression (fails on main, passes here).

---------

Signed-off-by: Snehil Shah <snehilshah.989@gmail.com>
Co-authored-by: felipe stival <14948182+v0idpwn@users.noreply.github.com>

Coverage Stats

54 of 54 new or added lines in 5 files covered. (100.0%)

2 existing lines in 2 files now uncovered.

2987 of 3667 relevant lines covered (81.46%)

50777.73 hits per line