toppk

> This pull request **introduces 2 alerts** when merging [6c98cef](https://github.com/cherrypy/cheroot/commit/6c98cef1e686f5dd38111175e543cab8d6243e9b) into [3915f35](https://github.com/cherrypy/cheroot/commit/3915f35fbfe34c7cd0e4ffe68aba092f750345ac) - [view on LGTM.com](https://lgtm.com/projects/g/cherrypy/cheroot/rev/pr-814ef145497e27488287d057ed4d6b19a972ff61) > > **new alerts:** thank you bot! > * 1 for Unreachable 'except' block...

> Well, it may cause hard-to-debug states if the exceptions are just ignored silently. The currently listed errors were carefully curated, and I'm not sure if it's reasonable to throw...

i'll take a look at your last recommendation and update this pr. not sure it will pass the complexity review, but we'll see.

I've done some digging, it seems that the sql that is blowing up is the introspection that is being run. It seems that CTEs aren't fully implemented in crdb, for...

Actually, in my case it's okay not to avoid using prepared parameter for the enum type, while still using prepared parameters for the rest of the arguments. i'm using prepared...

> Could we please get this merged? @toppk Are you still planning on pushing this PR to completion? I am willing to, but I'm having a hard time incorporating the...

That would work quite well. I guess I misunderstood the code, because it looked to me as if the recv() method is decoupled from where the actual processing of inbound...

just thinking about the send side, I think it really is less important. there aren't too many servers that are strict in what they accept, especially when they are expecting...

i compiled from master with debug, and recorded the debug info (printing in protocol.connection_lost()) after a few runs (you can tell from the closed handles, that I'm not posting each...

fwiw, running this after the transport is closed down seems to trigger a cleanup ```python import ctypes def malloc_trim(): ctypes.CDLL("libc.so.6").malloc_trim(0) ``` as a workaround, I'm planning on firing a call...