Just dropping by to see if there's any concerns about this PR.

I took a quick look.
Gonna run our CI/CD checks on this.
My first thought is that because we are adding a couple code pathscoverage is likely to fail.
Also curious why the pragma forqueue.full: we recently got bitten by having a code branch that was not tested and I am hesitant about adding more such paths to the code base without understanding for myself why it is necessary/desireable.

Thequeue.Full code path is hard to test because it's non-deterministic: the queue can be full or not depending on the timing of consumers and producers.

So, I just put apragma there to ignore the coverage, similar to the existing code a few lines below.

Looks like cover is passing.

After sleeping on this, I do think it's important to try and cover the currently uncovered branch. I'll see what I can do in a follow-up PR today.

Edit: mailed#2127