-
Notifications
You must be signed in to change notification settings - Fork 491
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
Remove the tests for pg_anon
#10382
Remove the tests for pg_anon
#10382
Conversation
347d2dd
to
6c67546
Compare
7359 tests run: 6979 passed, 0 failed, 380 skipped (full report)Flaky tests (8)Postgres 17
Postgres 16
Postgres 15
Code coverage* (full report)
* collected from Rust tests only The comment gets automatically updated with the latest test results
7daf17a at 2025-01-21T16:39:25.189Z :recycle: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we remove the extension itself?
Let's try. |
Are we sure that there are no users using pg_anon at the moment? |
See https://github.com/neondatabase/cloud/issues/22663 |
Sounds like we just have one user left. Is someone reaching out to that user? |
d251e9f
to
7daf17a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are we making changes in the compute dockerfile?
We have to have the sources of the extension to run the tests. |
Problem
We are removing the
pg_anon
v1 extension from Neon. So we don't need to test it anymore and can remove the code for simplicity.Summary of changes
The code required for testing
pg_anon
is removed.