Skip to content

Conversation

@darora
Copy link
Contributor

@darora darora commented Jul 15, 2021

Built on top of PG13 refactor, but doesn't need to bundled in as part of the v0 image - let's get that out the door first.

@darora darora requested review from awalias, dragarcia and inian July 15, 2021 04:59
state: stopped
with_items:
- snapd
- postgresql
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for postgresql, correct me if I'm mistaken but I believe we would need it to have it started since we're running the .sql scripts at boot (then restarting it)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah right, yeah

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Going to leave it stopped per offline discussion; we can start it up once it's been configured, before the init scripts need to be run.

with_items:
- snapd
- postgresql
- postgrest
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this be pgbouncer instead?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added pgbouncer; this list is a superset of the things that might be enabled in the image

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(pgbouncer isn't enabled)

@dragarcia dragarcia merged commit 1ba508f into feature/supabasePostgresV1.0 Jul 15, 2021
@dragarcia dragarcia deleted the da/boot-time-optimizations branch July 15, 2021 08:21
damonrand pushed a commit to cepro/postgres that referenced this pull request Jun 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants