supavisor icon indicating copy to clipboard operation
supavisor copied to clipboard

AppSheet + Supabase: Since new Pooler, the connection is basically impossible to work with

Open SkrOYC opened this issue 1 year ago • 48 comments

Bug report

Describe the bug

When using port "5432" things work with the obvious limits. AppSheet (a no-code platform owned by Google) opens a connection for each CRUD operation and this was solved by using the pooler on port "6543". Everything worked perfect untill PgBouncer was left aside. It justs seems using the pooler's port is not a transparent solution from the client's side. Since this is a platform not managed by me I don't know the ins and outs of it, but I can say this was working perfectly before with PgBouncer.

To Reproduce

Use AppSheet and connect to the database using the Postgres connector with the 5432 port. Everything works perfect. Change it to 6543 and you will face timeout issues. I can't add tables to my apps nor sync data on others apps already working with 5432.

Expected behavior

That the connection to the database under 6543 should work the same way 5432 does, being transparent on the client's side.

Additional info

This is a post in Appsheet's official community I made to promote Suabase and where users have been reporting the issue, which I was able to confirm https://www.googlecloudcommunity.com/gc/Tips-Tricks/Updated-Supabase-mets-AppSheet/m-p/442135/highlight/true#M6685

SkrOYC avatar Apr 22 '24 18:04 SkrOYC

Thanks for opening! To confirm, you have 3 options to connect with AppSheet:

  1. Direct - 5432 - This is a direct connection to the database
  2. Pooler - Transaction - 6543 - This is a connection to your database through the Pooler where the connection is opened only for the lifetime of a transaction
  3. Pooler - Session - 5432 - This is a connection to your database through the Pooler where the connection is open until the client disconnects (so you are likely to hit connection limits here)

AppSheet may recommend connection types but I would typically go through the Pooler's transaction (and modify timeouts if needed) rather than going through session as the client may keep connections open longer than needed so they are not freed for others.

Read more here

Let us know if that answers your question!

encima avatar Apr 23 '24 07:04 encima

That's kinda what I already knew about it. The issue in this case is that I need transaction mode and it just doesn't work at all

SkrOYC avatar Apr 23 '24 12:04 SkrOYC

"Doesn't work at all" - Do you have more info on this? What are the logs on your instance reporting for the timeouts? What is the instance size and pool size you have configured?

Your Expected behaviour section expects Transaction mode to work the same as a direct connection but this is not the case. Session mode works the same as a direct connection.

encima avatar Apr 23 '24 14:04 encima

What I mean to what I expect is that from the client side we should see things working since Transaction mode has benefits on top of what Session mode can offer. I can connect using Session mode but if I change to port 6543 to not have issues with open connections (since the platform I'm using doesn't close them) it doesn't do anything other than authenticating. I can't sync data. I could do some troubleshooting if you tell me where I should find the logs for the sync attempts

SkrOYC avatar Apr 23 '24 14:04 SkrOYC

Noted, thanks for the extra info. I have a gut feeling that the outcome of this will be: "AppSheet does not support transaction mode when connecting to Postgres"

But, I am not sure so let's confirm this: you can check your Pooler logs in the dashboard under Logs by clicking Pooler.

encima avatar Apr 23 '24 14:04 encima

About the your first comment, my only knowledge about the way AppSheet works under the hood is that it was working wonderfully before with PgBouncer. I expect it to work as good as before with Supavisor but for some reason it doesn't. I though Supavisor was supposed to be a drop in replacement for PgBouncer. If there is a difference in the way Supavisor exposed the database in comparison to PgBouncer, that should be the issue.

I'll do some troubleshooting and post the logs here.

Thanks for your help!

SkrOYC avatar Apr 23 '24 14:04 SkrOYC

Hey @SkrOYC

That is a good point and I am not sure if the switch was called out as pgbouncer was set to transaction mode by default. When you say AppSheet worked with pgBouncer was that after changing to session mode as well?

encima avatar Apr 24 '24 05:04 encima

Closing due to inactivity

encima avatar May 09 '24 03:05 encima

Finding time to keep troubleshooting. This is still an issue. In the meantime, I think we need to confirm from the people that was tied to the supavisor implementation if there is any difference in the way it behaves compared to pgbouncer that may make this not a 1:1 replacement to pgbouncer

SkrOYC avatar May 10 '24 18:05 SkrOYC

Anyone has any idea about why the new Supavisor integration was not a 1:1 replacement to pgBouncer?

SkrOYC avatar May 29 '24 18:05 SkrOYC

Btw @encima

When you say AppSheet worked with pgBouncer was that after changing to session mode as well?

AppSheet worked perfectly with pgBouncer in Transaction mode, CRUD operations were running quickly

SkrOYC avatar May 29 '24 18:05 SkrOYC

"ClientHandler: socket closed with reason {:shutdown, :socket_closed}"

Same message again and again in the logs

SkrOYC avatar May 29 '24 18:05 SkrOYC

I just found this message in the dashboard image

Is it possible that the change to Supavisor is half done or it's expecting pgBouncer and the infrastructure is not there anymore? @encima

SkrOYC avatar May 29 '24 18:05 SkrOYC

Also found some of these under the "Postgres" logs: image

SkrOYC avatar May 29 '24 18:05 SkrOYC

@SkrOYC You likely need to upgrade your instance. pgbouncer is not enabled on (almost) all instances but the user still exists in the DB. "pending removal" means that the disabled pgbouncer will be removed completely from future instances but is not currently
To confirm, the switch to supavisor is complete but the phasing out of pgbouncer is not. Supavisor is not a 1:1 replacement (both are open source so you are welcome to compare) but you can expect similar behaviours, if not the same. You can add "pgbouncer=true" to your Supavisor connection string to make the behaviour more similar which will disable prepared statements.

encima avatar May 29 '24 18:05 encima

You likely need to upgrade your instance

Any clue on how to deal with it?

Also, we are connecting through the AppSheet connector for it, which takes the host:port, database name, user, password and nothing else

SkrOYC avatar May 29 '24 18:05 SkrOYC

@SkrOYC you can see upgrades in the "Infrastructure" section in your project settings.

Ah, in that case you can use Supavisor transaction mode, session mode or a direct connection to the DB

encima avatar May 29 '24 18:05 encima

in that case you can use Supavisor transaction mode, session mode or a direct connection to the DB

That's what we are trying to do, and worked perfectly before under pgBouncer and now it doesn't

you can see upgrades in the "Infrastructure" section in your project settings.

I can't see any upgrade options btw @encima

SkrOYC avatar May 30 '24 18:05 SkrOYC

I can't see any upgrade options btw @encima That should mean you are all good!

I see what you mean, now, sorry, I think I was confusing this with a different case.

Direct connections do work but I can reproduce your issue when using Supavisor and enforcing SSL

So, I can connect when using Supavisor, disabling SSL enforcement and not requiring SSL in AppSheet.

I will check with the pooler team if they can investigate more!

encima avatar May 31 '24 08:05 encima

I can reproduce your issue when using Supavisor and enforcing SSL

SSL doesn't work but that's not a big issue for the time being, it's an AppSheet thing.

I can connect when using Supavisor, disabling SSL enforcement and not requiring SSL in AppSheet.

So can I, but after that I cannot add any table to any app nor sync anything on apps that were already working

SkrOYC avatar May 31 '24 20:05 SkrOYC

I tried with a fresh database, so no "pgBouncer pending removal" issue. Transaction mode, new app inside AppSheet instead of an already working one and no luck. It doesn't work. image These are the logs when I tried to connect

SkrOYC avatar Jun 25 '24 17:06 SkrOYC

Transferring to the Supavisor repo so the @supabase/pooler team are aware

encima avatar Jul 08 '24 09:07 encima

Hi everyone. I have a client wanting to use Supabase but still laking a solution on this

SkrOYC avatar Jul 26 '24 05:07 SkrOYC

Checking in with the Pooler team, @SkrOYC, thank you for the ping!

encima avatar Jul 29 '24 11:07 encima

could i know how long will it take to help resolve this issue?

Khoanguyen0109 avatar Jul 29 '24 15:07 Khoanguyen0109

Difficult to estimate, we will update this issue with any updates so just be sure to subscribe for notifications!

encima avatar Jul 29 '24 16:07 encima

Difficult to estimate, we will update this issue with any updates so just be sure to subscribe for notifications!

Have you fixed this bug yet?

NguyenHoadev avatar Jul 31 '24 14:07 NguyenHoadev

??????????????????????????????

NguyenHoadev avatar Aug 10 '24 18:08 NguyenHoadev

Hi @encima. Is there anyone in particular in the Supavisor team that could help us with testing? This has been left aside for months. Thanks

SkrOYC avatar Aug 19 '24 21:08 SkrOYC

Hey @SkrOYC The pooler team is aware of this issue and it is currently in their backlog

encima avatar Aug 20 '24 00:08 encima