Skip to content

Log PostgreSQL metastore error #5530

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
Merged
merged 1 commit into from
Oct 29, 2024
Merged

Log PostgreSQL metastore error #5530

merged 1 commit into from
Oct 29, 2024

Conversation

guilload
Copy link
Member

Description

Per title.

How was this PR tested?

Ran quickwit-metastore test suite with the postgres feature enabled and checked the logging output.

@github-actions GitHub Actions
Copy link

On SSD:

Average search latency is 0.999x that of the reference (lower is better).
Ref run id: 4045, ref commit: 105aa7d
Link

On GCS:

Average search latency is 0.949x that of the reference (lower is better).
Ref run id: 4046, ref commit: 105aa7d
Link

@guilload guilload merged commit 826f10f into main Oct 29, 2024
5 checks passed
@guilload guilload deleted the guilload/improve-pg-metastore-logging branch October 29, 2024 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@trinity-1686a trinity-1686a trinity-1686a approved these changes

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
@guilload @trinity-1686a