Results 28 comments of Mostafa

I managed to get a debugging environment going and localized the error to `clstmhl.h` line 107 The saved model has a layer of the kind "NPLSTM_SigmoidTanhTanh" and apparently it is...

@nmldiegues I think I am running into the same issue and I am able to reproduce it. I tried your patch I am a getting panic ``` panicked at 'already...

Hey, thanks for reporting. That's a great find! I looked at the code and I think Pgcat handles it correctly. In this section https://github.com/levkk/pgcat/blob/main/src/client.rs#L578-L603 , Pgcat sends the cancel request...

Distributed Pgcat setups with multiple instances behind a load balancer are definitely affected by this bug because the secret keys are not shared between Pgcat instances.

I apologize, I was responding to the issue you described in the video 😓 Yep. that race condition definitely exists in Pgcat.

I am looking at this. I am able to reproduce this on the latest version of PgCat. It seems to be somehow related to the min_pool setting because when I...

Well, that was easy (it is actually in the method name :sweat_smile: ) https://github.com/postgresml/pgcat/blob/2def40ea6ae489ccd6e5dd2729e4ee7f41e7e757/src/pool.rs#L480 bb8 docs for `min_idle` states that ``` /// Sets the minimum idle connection count maintained by...

Reading through the PR that introduced this, it seems that Lev knew about this behavior https://github.com/postgresml/pgcat/pull/416 So maybe this is a feature not a bug? The name of the setting...

I haven't had the time to review this yet. I will take a look later this week or on the weekend.

I will appreciate it if you could add some tests. I'll update the `CONTRIBUTING` file to show how you can add tests in Ruby/Python/Go/Rust and how you can run them...