-
Notifications
You must be signed in to change notification settings - Fork 888
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
PG16 - Adapt to the new RTEPerminfo logic. #6072
PG16 - Adapt to the new RTEPerminfo logic. #6072
Conversation
@mahipv, @konskov: please review this pull request.
|
945417b
to
ad1b470
Compare
Couple more PG16 patches in https://github.com/lkshminarayanan/timescaledb/commits/pg16-fixes if anyone is interested. |
Codecov Report
@@ Coverage Diff @@
## main #6072 +/- ##
==========================================
- Coverage 81.53% 81.48% -0.06%
==========================================
Files 246 246
Lines 56782 56739 -43
Branches 12588 12569 -19
==========================================
- Hits 46297 46232 -65
- Misses 8103 8117 +14
- Partials 2382 2390 +8
... and 21 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
760b7dd
to
d0ee516
Compare
Thanks a lot @lkshminarayanan ... we'll take care of it and add you as author |
Sure! Thank you @fabriziomello. |
356ccfb
to
7b8e361
Compare
7b8e361
to
f7acf86
Compare
PG16 moves the permission checking information out of the range table entries into a new struct - RTEPermissionInfo. This commit updates timescaledb to use the this new mechanism when compiling with PG16. postgres/postgres@a61b1f74 postgres/postgres@b803b7d1 postgres/postgres@47bb9db7
f7acf86
to
3f612f0
Compare
Merged it and will send a following PR with some fixes on regression tests suite. |
With the two commits included in this PR, timescaledb can be compiled with PG16.
Disable-check: force-changelog-file
Disable-check: commit-count