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

Binary size + composability #1255

Open
gatesn opened this issue Nov 8, 2024 · 0 comments
Open

Binary size + composability #1255

gatesn opened this issue Nov 8, 2024 · 0 comments

Comments

@gatesn
Copy link
Contributor

gatesn commented Nov 8, 2024

As flagged by @XinyuZeng

Right now it's not possible to strip out unused default encodings from Vortex. For example, if we only include PrimitiveArray in the Context, then many compute function implementations get pulled in from VarbinArray, even with LTO. It's also not clear that we would strip out unused compute functions of used encodings.

This issue is to track a solution to this, likely involving re-architecting our crates / traits.

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

No branches or pull requests

1 participant