Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update docker-entrypoint.sh #979

Closed
wants to merge 2 commits into from

Conversation

Cookieees
Copy link

I just made the same changes on my own and got adequate, expected and logical behavior. I don't mean to be rude, but I just want to ask: why aren't these changes still in the original postgres image?
After all, this is a logical behavior - the user, if he needs it, adds his always-init.sh script to the always-init directory, if not needed - he does not add anything and postgres simply does nothing "always" (regardless of external conditions).

I just made the same changes on my own and got adequate, expected and logical behavior. I don't mean to be rude, but I just want to ask: why aren't these changes still in the original postgres image?
After all, this is a logical behavior - the user, if he needs it, adds his always-init.sh script to the always-init directory, if not needed - he does not add anything and postgres simply does nothing "always" (regardless of external conditions).
Only the duplication of the start/stop temp_server has been eliminated
@tianon
Copy link
Member

tianon commented Jul 12, 2022

Unfortunately, starting the server temporarily like this can be very expensive, especially for large databases, which is why we don't/can't do it this way by default. 😬

(See also #173, #821, #191, #929, #955)

@tianon tianon closed this Jul 12, 2022
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.

2 participants