Skip to content

disable arrow compression - #775

Merged
bors[bot] merged 7 commits into
masterfrom
disable-arrow-compression
Apr 14, 2023
Merged

bors[bot] merged 7 commits into
masterfrom
disable-arrow-compression

Conversation

@jdroenner

Copy link
Copy Markdown
Member
  • I added an entry to CHANGELOG.md if knowledge of this change could be valuable to users.

Here is a brief summary of what I did:

I disabled the compression in raster streams because pyarrow (using Arrow C++) has this issue: apache/arrow#15102

@jdroenner

Copy link
Copy Markdown
Member Author

@coveralls

Copy link
Copy Markdown
Collaborator

Pull Request Test Coverage Report for Build 4670058238

  • 30 of 30 (100.0%) changed or added relevant lines in 2 files are covered.
  • 5 unchanged lines in 4 files lost coverage.
  • Overall coverage decreased (-0.05%) to 89.304%

Files with Coverage Reduction New Missed Lines %
services/src/datasets/external/aruna/mock_grpc_server.rs 1 89.02%
services/src/pro/permissions/postgres_permissiondb.rs 1 66.13%
services/src/pro/users/postgres_userdb.rs 1 95.96%
services/src/pro/contexts/postgres.rs 2 96.51%
Totals Coverage Status
Change from base Build 4670047823: -0.05%
Covered Lines: 94450
Relevant Lines: 105762

💛 - Coveralls

@michaelmattig

Copy link
Copy Markdown
Contributor

bors merge

@bors
bors Bot merged commit d4329aa into master Apr 14, 2023
@bors
bors Bot deleted the disable-arrow-compression branch April 14, 2023 10:57
github-merge-queue Bot pushed a commit that referenced this pull request Apr 17, 2026
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.

3 participants