On Fri, Jul 18, 2025 at 11:11 AM Jacob Champion <jacob.champ...@enterprisedb.com> wrote: > The attached still needs some documentation work
v2 does a bunch of commit message work, but I imagine it needs a good bit of copy-editing for clarity. I'm not in any hurry to smash this in. I think we still need - independent verification of the architectural issue, to make sure it's not any deeper or shallower than pqReadData() - independent verification that this fixes the bugs that have been described - measurement of the performance characteristics of the new code - verification of the maximum amount of additional buffer memory that can be consumed during the drain - consensus that we want to maintain this new behavior - discussion of what we want this code to look like going forward Andres, does this patch help clarify my thoughts upthread? Ideally the additional code isn't getting in the way of any future rearchitectures, since it only pins the new requirement in the code that needs it. Thanks, --Jacob
v2-0001-libpq-Extend-read-pending-check-from-SSL-to-GSS.patch
Description: Binary data
v2-0002-libpq-Drain-all-pending-bytes-from-SSL-GSS-during.patch
Description: Binary data