Skip to content

Conversation

@fabriziomello
Copy link
Member

@fabriziomello fabriziomello self-assigned this Oct 29, 2025
@fabriziomello fabriziomello force-pushed the pg18_use_pg_module_magic_ext branch from 6575386 to 85d718b Compare October 29, 2025 20:32
@fabriziomello fabriziomello changed the title Use PG_MODULE_MAGIC_EXT on PG18 Use PG_MODULE_MAGIC_EXT for PG18 Oct 29, 2025
@codecov
Copy link

codecov bot commented Oct 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.57%. Comparing base (ff980e6) to head (c5d4e87).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8868      +/-   ##
==========================================
+ Coverage   82.53%   82.57%   +0.04%     
==========================================
  Files         249      249              
  Lines       48286    48260      -26     
  Branches    12318    12314       -4     
==========================================
- Hits        39855    39853       -2     
- Misses       3494     3524      +30     
+ Partials     4937     4883      -54     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@fabriziomello fabriziomello force-pushed the pg18_use_pg_module_magic_ext branch from 85d718b to 6f23268 Compare October 30, 2025 12:48
@fabriziomello fabriziomello force-pushed the pg18_use_pg_module_magic_ext branch from 6f23268 to c5d4e87 Compare November 14, 2025 17:18
@fabriziomello fabriziomello marked this pull request as ready for review November 14, 2025 18:36
#if PG18_GE
#ifdef PG_MODULE_MAGIC_EXT
#define TS_MODULE_MAGIC \
PG_MODULE_MAGIC_EXT(.name = "timescaledb", .version = TIMESCALEDB_VERSION_MOD)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we have 3 different modules, should they all use the same name?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

loader version may not match timescaledb version at runtime which is expected feature

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants