Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Revert "pg_repack.c" #351

Merged
merged 1 commit into from
Jun 14, 2023
Merged

Revert "pg_repack.c" #351

merged 1 commit into from
Jun 14, 2023

Conversation

MichaelDBA
Copy link
Collaborator

Reverts #342

MichaelDBA
MichaelDBA commented Apr 23, 2023
View reviewed changes
Copy link
Collaborator Author

@MichaelDBA MichaelDBA left a comment

Choose a reason for hiding this comment

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

Apparently, I'm not following a consistent pattern here for logging warnings or errors, so I'm pulling the request.

@andreasscherbaum andreasscherbaum merged commit 54ad4f7 into master Jun 14, 2023
@andreasscherbaum
Copy link
Collaborator

We discussed this in our review meeting, and decided that this deserves a fix along the lines @za-arthur pointed out here.

@za-arthur za-arthur deleted the revert-342-master branch October 18, 2023 16:23
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MichaelDBA @andreasscherbaum