Skip to content

Conversation

heimann
Copy link
Contributor

@heimann heimann commented Dec 21, 2017


  • The page (if new), does not already exist in the repo.

  • The page (if new), has been added to the correct platform folder:
    common/ if it's common to all platforms, linux/ if it's Linux-specific, and so on.

  • The page has 8 or fewer examples.

  • The PR is appropriately titled:
    <command name>: add page for new pages, or <command name>: <description of changes> for pages being edited

  • The page follows the contributing guidelines

Copy link
Member

@sbrl sbrl left a comment

Choose a reason for hiding this comment

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

Hey, thanks for the new page, @heimann! Looks ok to me 😺

Copy link
Collaborator

@felixonmars felixonmars left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@felixonmars felixonmars added the new command Issues requesting creation of a new page or PRs adding a new page for a command. label Dec 27, 2017
@sbrl sbrl merged commit 22042bb into tldr-pages:master Dec 27, 2017
@sbrl
Copy link
Member

sbrl commented Dec 27, 2017

Thanks, @heimann! 😺


- Stop a PostgreSQL server:

`pg_ctl -D {{data_directory}} stop`
Copy link
Member

Choose a reason for hiding this comment

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

Don't we need to be root for the start/stop operations ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, I think it's best practice to not run those as root. If you initialize the db and root owns the data directory then yes you need to be root to stop it, but the recommended way is to actually create a separate 'postgres' user and run it as that user:
su postgres -c 'pg_ctl start -D /usr/local/pgsql/data -l serverlog'
as per: https://www.postgresql.org/docs/9.1/static/server-start.html

Copy link
Member

Choose a reason for hiding this comment

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

Right, thanks.

@dmmqz dmmqz mentioned this pull request Sep 7, 2025
35 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new command Issues requesting creation of a new page or PRs adding a new page for a command.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants