Skip to content
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

error: redefinition of ‘RelationGetSmgr’ #6709

Closed
anthonator opened this issue Feb 9, 2023 · 1 comment
Closed

error: redefinition of ‘RelationGetSmgr’ #6709

anthonator opened this issue Feb 9, 2023 · 1 comment

Comments

@anthonator
Copy link

When building Citus 11.2.0 on PostgreSQL 13.10 and 14.7 we're receiving the following errors.

/usr/local/src/citus/src/include/pg_version_compat.h:59:1: error: redefinition of ‘RelationGetSmgr’
   59 | RelationGetSmgr(Relation rel)
      | ^~~~~~~~~~~~~~~
In file included from /usr/include/postgresql/14/server/access/tableam.h:24,
                 from /usr/local/src/citus/src/include/columnar/columnar_tableam.h:7,
                 from columnar_customscan.c:46:
/usr/include/postgresql/14/server/utils/rel.h:544:1: note: previous definition of ‘RelationGetSmgr’ was here
  544 | RelationGetSmgr(Relation rel)
      | ^~~~~~~~~~~~~~~
make[1]: *** [<builtin>: columnar_customscan.o] Error 1
make[1]: Leaving directory '/usr/local/src/citus/src/backend/columnar'
make: *** [Makefile:19: columnar] Error 2

We do not see this error on PostgreSQL 15.2.

@JelteF
Copy link
Contributor

JelteF commented Feb 10, 2023

Our own backport of that function conflicted with the backport that was done by postgres in this release: postgres/postgres@32d5a49

I fixed this issue in #6711 and I backported the change to our 11.2 branch: 341fdb3. So when we tag the next 11.2 patch release, you can use that one. For now you can probably use the specific commit.

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

No branches or pull requests

2 participants