Inefficient use of Flush in extended query protocol
Nobody has claimed this yet.
- Dominant language
- Scala
- Stars
- 1.7k
- Forks
- 175
- Avg merge
- 5d 14h
- Merged PRs (30d)
- 9
Description
When executing a prepared statement, Skunk currently uses the following protocol flow:
- Parse
- Flush
- Await response and check for errors
- Describe statement
- Flush
- Await response and check for errors
- Bind
- Flush
- Await response and check for errors
- Execute
- Flush
- Await response and check for errors
- Close portal
- Flush
- Await response and check for errors
- Close statement
- Flush
- Await response and check for errors
This is needlessly inefficient, as it prevents batching of request commands and requires a full network round-trip on every step.
In addition, this currently fails to work—and instead hangs indefinitely—with a Pgpool-II server, which does not fully honour Flush. (This is currently being discussed on the pgpool-general mailing list.)
As far as I can tell, the only reason it is done this way is that MessageSocket does not allow queueing up response processing asynchronously and submitting more commands in the meantime. For example, skunk.net.protocol.Parse currently does this:
where flatExpect suspends the fibre handling the current database connection until a response message is received. In an ideal world, it would schedule the block to be run whenever the next message arrives, but continue executing the sending fibre so more protocol commands can be queued; so the Flush could be omitted.
Any idea as to how this could be allowed in a clean way? I’m thinking that the sending and receiving should be separate fibres, but I’m not well-versed enough to see immediately how to achieve that.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the linked Parse.scala section and inspect MessageSocket, especially how flatExpect suspends response handling. Trace the prepared-statement protocol flow and determine how request batching can avoid per-step Flush calls while still processing responses; done includes avoiding the Pgpool-II hang.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- postgres, scala
- Domain
- databases, networking
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 25/100