Skip to content

Build failed with Postgres 17 #146

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
shuaitian-git opened this issue Feb 26, 2025 · 3 comments
Open

Build failed with Postgres 17 #146

shuaitian-git opened this issue Feb 26, 2025 · 3 comments

Comments

@shuaitian-git
Copy link

Need to refactor the code towards PG17

Error message:

src/rumsort.c: In function ‘rum_tuplesort_putrum’:
src/rumsort.c:496:9: error: too few arguments to function ‘tuplesort_puttuple_common’
496 | tuplesort_puttuple_common(state, &stup, false);
| ^~~~~~~~~~~~~~~~~~~~~~~~~
In file included from /usr/lib/postgresql/17/include/server/nodes/execnodes.h:49,
from /usr/lib/postgresql/17/include/server/executor/execdesc.h:18,
from /usr/lib/postgresql/17/include/server/executor/executor.h:17,
from src/rumsort.c:23:
/usr/lib/postgresql/17/include/server/utils/tuplesort.h:379:13: note: declared here
379 | extern void tuplesort_puttuple_common(Tuplesortstate *state,
| ^~~~~~~~~~~~~~~~~~~~~~~~~
src/rumsort.c: In function ‘rum_tuplesort_putrumitem’:
src/rumsort.c:514:9: error: too few arguments to function ‘tuplesort_puttuple_common’
514 | tuplesort_puttuple_common(state, &stup, false);
| ^~~~~~~~~~~~~~~~~~~~~~~~~
In file included from /usr/lib/postgresql/17/include/server/nodes/execnodes.h:49,
from /usr/lib/postgresql/17/include/server/executor/execdesc.h:18,
from /usr/lib/postgresql/17/include/server/executor/executor.h:17,
from src/rumsort.c:23:
/usr/lib/postgresql/17/include/server/utils/tuplesort.h:379:13: note: declared here
379 | extern void tuplesort_puttuple_common(Tuplesortstate *state,
| ^~~~~~~~~~~~~~~~~~~~~~~~~
make: *** [: src/rumsort.o] Error 1
Error on or near line 44: x; exiting with status 1
Error: Process completed with exit code 1.

@sokolcati
Copy link
Contributor

Hi!
RUM was updated to 17 PostgreSQL in commit cb1edffc5773
Please, use tag 1.3.14 (or source code from master branch)

@shuaitian-git
Copy link
Author

thanks, problem solved after updating to 1.3.14, could you create a new release for it?

@sokolcati
Copy link
Contributor

It's not problem. But I'm curious - why is a release preferred over a tag?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants