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

Add compression functions, features, docs, tests #8

Merged
merged 1 commit into from
Apr 18, 2024
Merged

Conversation

jasonmp85
Copy link
Contributor

@jasonmp85 jasonmp85 commented Apr 18, 2024

This adds a dependency on Hydra columnar and pg_cron.

Tests were all working prior to adding the columnar dependency. Getting it on the test machine may prove annoying.s

@jasonmp85 jasonmp85 force-pushed the compression branch 3 times, most recently from 9abd4f4 to 12de202 Compare April 18, 2024 14:57
@ChuckHend
Copy link
Member

ChuckHend commented Apr 18, 2024

Could it be installed using trunk? Here is an example of pg_cron being installed. Maybe that would work for columnar too?

@jasonmp85
Copy link
Contributor Author

Could it be installed using trunk? Here is an example of pg_cron being installed. Maybe that would work for columnar too?

I avoided it previously because using the apt repositories was faster, but Hydra obscures where it gets its deb from this pgxman thing. I'll switch to trunk.

@jasonmp85 jasonmp85 force-pushed the compression branch 3 times, most recently from 513cc3e to 1e9b0f7 Compare April 18, 2024 15:18
This adds a dependency on Hydra columnar and pg_cron.
@jasonmp85
Copy link
Contributor Author

Will add binary caching in a bit, but tests passing on all PG versions, even though adding Hydra makes local dev annoying (no Mac build), but will deal with that later.

@jasonmp85 jasonmp85 merged commit 0df1853 into main Apr 18, 2024
4 checks passed
@jasonmp85 jasonmp85 deleted the compression branch April 18, 2024 15:27
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