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

perf: have syscall/datastore insert method avoid converting the BSATN-encoded row to PV #2014

Open
Centril opened this issue Nov 27, 2024 · 1 comment
Assignees

Comments

@Centril
Copy link
Contributor

Centril commented Nov 27, 2024

Depends on #2013.

Deal with the BSATN directly, inserting it into the table, avoiding the PV.

Indices will still need AVs, but will be projected from the BFLATN.

@cloutiertyler
Copy link
Contributor

https://discord.com/channels/931210784011321394/1150256030433955870/1311406963208359996

Discussion regarding whether it's actually prereq. TL;DR we're going to add a new insert_bsatn function in write_row_to_pages_bsatn and then remove the insert function in a separate PR while updating the merge function in that PR as well.

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

2 participants