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

xsd:hexBinary typed literals appear to be stored as actual strings #123

Open
ebremer opened this issue Feb 12, 2021 · 4 comments
Open

xsd:hexBinary typed literals appear to be stored as actual strings #123

ebremer opened this issue Feb 12, 2021 · 4 comments

Comments

@ebremer
Copy link
Contributor

ebremer commented Feb 12, 2021

xsd:hexBinary typed literals appear to be stored as an actual string rather than converted to binary. This seems to be an opportunity to get further compression.

@ebremer
Copy link
Contributor Author

ebremer commented Feb 12, 2021

Same seems to be true for xsd:base64Binary

@mielvds
Copy link
Member

mielvds commented Mar 15, 2021

Fair point! The gain would be minimal though, unless you have a bunch of those. Would you mind submitting a PR?

@ebremer
Copy link
Contributor Author

ebremer commented Mar 18, 2021

I would have a quite a bunch. Does HDT's design allow for (if coded up) storage of the hexbins and base64's as actually binary data? If so, I would pursue it if it would be a welcome modification.

@mielvds
Copy link
Member

mielvds commented Mar 19, 2021

It would! But it would also introduce a breaking change in the HDT files, meaning we'd have to move up a version and also reflect the changes in the C++ version. Not sure if that's worth it ATM without other improvements to release. That said, a branch is always welcome, ready to be included in a next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants