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.

Sign up for GitHub

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

Jump to bottom

growing_tape_t can't handle NULL entries #90

Closed
ashvardanian opened this issue Sep 21, 2022 · 1 comment
Closed

growing_tape_t can't handle NULL entries #90

ashvardanian opened this issue Sep 21, 2022 · 1 comment
Assignees
@VioletaStepanyan
Milestone
v0.4: Sequenced T...

Comments

@ashvardanian
Copy link
Contributor

The lengths_ would have to be appended by ukv_length_missing_k.
A presences_ array must be added to contain validity bitmasks.
@VioletaStepanyan VioletaStepanyan mentioned this issue Sep 23, 2022
Merged
@ashvardanian
Copy link
Contributor Author

The value_view_t growing_tape_t::push_back(value_view_t value, ukv_error_t*) must support value = {}.

ashvardanian added a commit that referenced this issue Sep 27, 2022
@ashvardanian
Fix: Exporting nulls into tape #90
f0769da
@ashvardanian ashvardanian added this to the 0.4.0 milestone Oct 3, 2022
DarvinHarutyunyan pushed a commit that referenced this issue Dec 9, 2022
growing_tape_t can't handle NULL entries #90
dda5d72
DarvinHarutyunyan pushed a commit that referenced this issue Dec 9, 2022
Recover: Usage: growing_tape_t can't handle NULL entries #90
70bd5f1
DarvinHarutyunyan pushed a commit that referenced this issue Dec 9, 2022
@ashvardanian
Fix: Exporting nulls into tape #90
4213317
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@VioletaStepanyan VioletaStepanyan

Labels
None yet
Projects
No open projects
UKV Community Edition
Status: Done
Milestone
v0.4: Sequenced Transactions, Parquet...
Development

No branches or pull requests
3 participants
@ashvardanian @davvard @VioletaStepanyan