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.

Sign up for GitHub

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

Jump to bottom

Suppress RAISE NOTICE #2

Merged
merged 2 commits into from
Jul 6, 2021
Merged

Suppress RAISE NOTICE #2

merged 2 commits into from
Jul 6, 2021

Conversation

za-arthur
Copy link
Contributor

@za-arthur za-arthur commented Jul 2, 2021 •
edited
Loading

mvtbl is pretty noisy. RAISE NOTICE logs message to stderr, therefore it is sent to dba@adjust.com. It clutters up cron alerts.

We can remove RAISE NOTICE safely since we already have logging in the script itself:
https://github.com/adjust/db_jobs/blob/3fab4a6e9ab789c34ef958fa86fc292f9c9ca710/backends/scripts/move_to_pg_default.sh#L66

@za-arthur za-arthur added the enhancement New feature or request label Jul 2, 2021
@za-arthur za-arthur self-assigned this Jul 2, 2021
@za-arthur za-arthur merged commit 79883b8 into master Jul 6, 2021
This was referenced Jul 6, 2021
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@scorpevans scorpevans scorpevans approved these changes

@rapimo rapimo Awaiting requested review from rapimo

@mrimbault mrimbault Awaiting requested review from mrimbault

Assignees

@za-arthur za-arthur

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@za-arthur @scorpevans