-
Notifications
You must be signed in to change notification settings - Fork 331
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 unused "transient" block adapter #7663
Conversation
It is unused. _Sic transit gloria mundi_, I guess.
9afd0df
to
930ae1f
Compare
There should never be any communication with a "transient" block adapter - it _literally_ ignores and fails everything. So this is not a breaking change, except perhaps for a system test that tests a client with a broken lakeFS server. We do not have such a test.
🎊 PR Preview af712e8 has been successfully built and deployed to https://treeverse-lakeFS-preview-pr-7663.surge.sh 🕐 Build time: 0.013s 🤖 By surge-preview |
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.
Looks OK, and I hope it really is 😉
Only thing blocking this is that we kinda break OpenAPI compatibility, because we remove the option to have "transient". It doesn't matter, but I'm also very willing to keep "transient" in swagger.yml and just make it an alias to |
This PR is now marked as stale after 30 days of inactivity, and will be closed soon. To keep it, mark it with the "no stale" label. |
Closing this PR because it has been stale for 7 days with no activity. |
It is unused. Sic transit gloria mundi, I guess.