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

Revisit bit_length Not Supporting Utf8View Array. #13329

Closed
austin362667 opened this issue Nov 9, 2024 · 2 comments · Fixed by #13538
Closed

Revisit bit_length Not Supporting Utf8View Array. #13329

austin362667 opened this issue Nov 9, 2024 · 2 comments · Fixed by #13538
Labels
bug Something isn't working

Comments

@austin362667
Copy link
Contributor

austin362667 commented Nov 9, 2024

Describe the bug

As mentioned in this (ongoing) PR, we have added utf8view support for the bit_length scalar function, as well as for the bit_length array function in this (merged) PR. So, this sql logic test error should be resolved after upgrading to the corresponding version of arrow-rs.

To Reproduce

No response

Expected behavior

query I
select bit_length(column1_utf8view) from test;
----
48
72
56
0
NULL

Additional context

Thanks for @goldmedal 's suggestion to track on this issue: #13221 (comment)

@austin362667 austin362667 added the bug Something isn't working label Nov 9, 2024
@goldmedal
Copy link
Contributor

I added #13336 to enhance the coverage. When solving this issue, we can move those tests to string_query.slt.part.

@austin362667
Copy link
Contributor Author

That's great! Thank you @goldmedal

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants