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

feat: Compression #245

Merged
merged 3 commits into from
Jul 21, 2023
Merged

feat: Compression #245

merged 3 commits into from
Jul 21, 2023

Conversation

disq
Copy link
Member

@disq disq commented Jul 19, 2023

Supports gzip compression by adding compression: gzip to spec.

@disq disq requested a review from yevgenypats July 19, 2023 12:29
@github-actions github-actions bot added the feat label Jul 19, 2023
@disq disq added the automerge Add to automerge PRs once requirements are met label Jul 21, 2023
@kodiakhq kodiakhq bot merged commit 9794651 into main Jul 21, 2023
4 checks passed
@kodiakhq kodiakhq bot deleted the feat/compression branch July 21, 2023 10:33
kodiakhq bot pushed a commit that referenced this pull request Jul 21, 2023
🤖 I have created a release *beep* *boop*
---


## [4.1.0](v4.0.3...v4.1.0) (2023-07-21)


### Features

* Compression ([#245](#245)) ([9794651](9794651))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
kodiakhq bot pushed a commit to cloudquery/cloudquery that referenced this pull request Jul 21, 2023
~Needs cloudquery/filetypes#245 to be tagged first~

- `{{FORMAT}}` now adds the compression format (ie. `.json` becomes `.json.gz`)

file plugin only:
- Updated tests to have separate temp dir for each test
- Since compressed data can't be read mid-write, using the custom test we use for parquet for compressed tests (which writes all data and then reads all back after closing)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Add to automerge PRs once requirements are met feat
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants