Georg Semmler

Thanks for providing these details. I think I now understand at least the problem that you are trying to solve. I'm still not sold on the implemented approach. > another...

Thanks for providing that maintained fork :heart: It might be worth to fill a rustsec advisory to mark the original crate as not maintained anymore and point to the fork...

I would like to raise the same semi related note as in #3639: As far as I remember one of the main motivations for the specification of diagnostic attribute namespaces...

Thanks for opening this PR and working on this issue. That written: I'm not fond of the chosen approach. I personally do not see why rebuilding the statement isn't an...

I'm open to allow configuring the cache, but I would like to make that option connection specific rather than global. So just `PgConnection::set_cache_size(CacheSize)` with cache size having 3 variants (`Unbound`...

That sounds like a good summary, thanks for writing it up. I think it would be also fine to have an additional function that just empties the current query cache.

Thanks for the update. I really like the solution to test this via the instrumentation tooling. It looks nearly ready now. I added a number of minor comments about stylistic...

@fraillt No worries, holiday is important and you shouldn't response to anything there. I already fixed the last few comments in https://github.com/diesel-rs/diesel/pull/4161/commits/bda514b4f88110a64d31f76bada2e81c599477a9, so there is nothing that needs to be...

Thanks for filling this. From playing a bit around on the psql command line, it seems like `array_append('{1, 2}', 3)[2]` is just not considered valid by postgresql. We can fix...

Closed by #4227