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
Use postgres implementation of estimate_hashagg_tablesize #2467
Use postgres implementation of estimate_hashagg_tablesize #2467
Conversation
4259951
to
a4d51b0
Compare
Codecov Report
@@ Coverage Diff @@
## master #2467 +/- ##
==========================================
+ Coverage 89.99% 90.16% +0.16%
==========================================
Files 213 212 -1
Lines 34406 34124 -282
==========================================
- Hits 30964 30768 -196
+ Misses 3442 3356 -86
Continue to review full report at Codecov.
|
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.
LGTM
src/compat.h
Outdated
* estimate_hashagg_tablesize is a static function in PG11 and earlier so we map | ||
* to our own copy when it's not available |
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.
* estimate_hashagg_tablesize is a static function in PG11 and earlier so we map | |
* to our own copy when it's not available | |
* estimate_hashagg_tablesize is a static function in PG11 and earlier, so we map | |
* to our own copy when it's not available. |
Only use our own implementation of estimate_hashagg_tablesize on versions where the postgres implementation is static.
a4d51b0
to
0e87833
Compare
Only use our own implementation of estimate_hashagg_tablesize on
versions where the postgres implementation is static.