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
Fix planner distributed table count #6113
Conversation
@gayyappan, @konskov: please review this pull request.
|
27149f1
to
6509392
Compare
The check for a distributed hypertable was done after ht had been changed to the compressed hypertable potentially leading to miscount or even segfault when the cache lookup for the compressed hypertable returned NULL.
6509392
to
c4b30c2
Compare
Codecov Report
@@ Coverage Diff @@
## main #6113 +/- ##
==========================================
+ Coverage 73.96% 81.51% +7.55%
==========================================
Files 246 246
Lines 49862 56730 +6868
Branches 12525 12568 +43
==========================================
+ Hits 36880 46244 +9364
- Misses 7128 8098 +970
+ Partials 5854 2388 -3466
... and 226 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Would it be possible to add a test for this fix? |
Hmm I think a test that actually fails without the patch is probably quite complex to setup. At least I dont know of a quick way. |
This release contains bug fixes since the 2.12.0 release. We recommend that you upgrade at the next available opportunity. **Bugfixes** * timescale#6113 Fix planner distributed table count * timescale#6117 Avoid decompressing batches using an empty slot * timescale#6123 Fix concurrency errors in OSM API * timescale#6142 do not throw an error when deprecation GUC cannot be read **Thanks** * @symbx for reporting a crash when selecting from empty hypertables
This release contains bug fixes since the 2.12.0 release. We recommend that you upgrade at the next available opportunity. **Bugfixes** * timescale#6113 Fix planner distributed table count * timescale#6117 Avoid decompressing batches using an empty slot * timescale#6123 Fix concurrency errors in OSM API * timescale#6142 Do not throw an error when deprecation GUC cannot be read **Thanks** * @symbx for reporting a crash when selecting from empty hypertables
This release contains bug fixes since the 2.12.0 release. We recommend that you upgrade at the next available opportunity. **Bugfixes** * timescale#6113 Fix planner distributed table count * timescale#6117 Avoid decompressing batches using an empty slot * timescale#6123 Fix concurrency errors in OSM API * timescale#6142 Do not throw an error when deprecation GUC cannot be read **Thanks** * @symbx for reporting a crash when selecting from empty hypertables
This release contains bug fixes since the 2.12.0 release. We recommend that you upgrade at the next available opportunity. **Bugfixes** * timescale#6113 Fix planner distributed table count * timescale#6117 Avoid decompressing batches using an empty slot * timescale#6123 Fix concurrency errors in OSM API * timescale#6142 Do not throw an error when deprecation GUC cannot be read **Thanks** * @symbx for reporting a crash when selecting from empty hypertables
This release contains bug fixes since the 2.12.0 release. We recommend that you upgrade at the next available opportunity. **Bugfixes** * #6113 Fix planner distributed table count * #6117 Avoid decompressing batches using an empty slot * #6123 Fix concurrency errors in OSM API * #6142 Do not throw an error when deprecation GUC cannot be read **Thanks** * @symbx for reporting a crash when selecting from empty hypertables
This release contains bug fixes since the 2.12.0 release. We recommend that you upgrade at the next available opportunity. **Bugfixes** * timescale#6113 Fix planner distributed table count * timescale#6117 Avoid decompressing batches using an empty slot * timescale#6123 Fix concurrency errors in OSM API * timescale#6142 Do not throw an error when deprecation GUC cannot be read **Thanks** * @symbx for reporting a crash when selecting from empty hypertables
This release contains bug fixes since the 2.12.0 release. We recommend that you upgrade at the next available opportunity. **Bugfixes** * #6113 Fix planner distributed table count * #6117 Avoid decompressing batches using an empty slot * #6123 Fix concurrency errors in OSM API * #6142 Do not throw an error when deprecation GUC cannot be read **Thanks** * @symbx for reporting a crash when selecting from empty hypertables
This release contains bug fixes since the 2.12.0 release. We recommend that you upgrade at the next available opportunity. **Bugfixes** * timescale#6113 Fix planner distributed table count * timescale#6117 Avoid decompressing batches using an empty slot * timescale#6123 Fix concurrency errors in OSM API * timescale#6142 Do not throw an error when deprecation GUC cannot be read **Thanks** * @symbx for reporting a crash when selecting from empty hypertables
This release contains bug fixes since the 2.12.0 release. We recommend that you upgrade at the next available opportunity. **Bugfixes** * timescale#6113 Fix planner distributed table count * timescale#6117 Avoid decompressing batches using an empty slot * timescale#6123 Fix concurrency errors in OSM API * timescale#6142 Do not throw an error when deprecation GUC cannot be read **Thanks** * @symbx for reporting a crash when selecting from empty hypertables
The check for a distributed hypertable was done after ht had been changed to the compressed hypertable potentially leading to miscount or even segfault when the cache lookup for the compressed hypertable returned NULL.
Fixes #5966