Skip to content

Conversation

@fabriziomello
Copy link
Member

Copy link
Member

@svenklemm svenklemm left a comment

Choose a reason for hiding this comment

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

Since this runs against main we dont need to keep previous api around here

We only need to add _timescaledb_functions.policy_compression_execute(job_id integer,htid integer,lag anyelement,maxchunks integer,verbose_log boolean,recompress_enabled boolean,use_creation_time boolean,amname name)

Since you also change it to variable its kinda hard to see what the actual change is.

@JamesGuthrie JamesGuthrie removed their request for review October 21, 2024 08:00
@fabriziomello fabriziomello force-pushed the sync_workflow_with_tsdb branch from 998e892 to 6b4aedd Compare October 21, 2024 13:05
@fabriziomello
Copy link
Member Author

Since this runs against main we dont need to keep previous api around here

We only need to add _timescaledb_functions.policy_compression_execute(job_id integer,htid integer,lag anyelement,maxchunks integer,verbose_log boolean,recompress_enabled boolean,use_creation_time boolean,amname name)

Since you also change it to variable its kinda hard to see what the actual change is.

Make sense. Fixed!

@fabriziomello fabriziomello merged commit f5bc5d8 into timescale:main Oct 22, 2024
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

Successfully merging this pull request may close these issues.

2 participants