-
Notifications
You must be signed in to change notification settings - Fork 1.1k
perf: improve calculating length performance for nested arrays in row conversion #9079
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
base: main
Are you sure you want to change the base?
perf: improve calculating length performance for nested arrays in row conversion #9079
Conversation
|
run benchmark row_format |
|
🤖 |
|
🤖: Benchmark completed Details
|
Which issue does this PR close?
N/A
Rationale for this change
Making the row length calculation faster which result in faster row conversion
What changes are included in this PR?
Rows(explained below)Are these changes tested?
Yes
Are there any user-facing changes?
Yes, added 3 functions to
Rows:row_len- get the row length at indexrow_len_unchecked- get the row length at index without bound checkslengths- get iterator over the lengths of the rowsRelated to:
GenericByteArrayin row conversion #9078