[ 
https://issues.apache.org/jira/browse/ARROW-13879?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17423460#comment-17423460
 ] 

Eduardo Ponce edited comment on ARROW-13879 at 10/2/21, 4:03 AM:
-----------------------------------------------------------------

Well, the issue is that `string_view` is used to encapsulate binary data in 
certain parts and [`string_view.length()` is used to get the size but it stops 
at the first null 
byte|https://github.com/apache/arrow/blob/master/cpp/src/arrow/vendored/string_view.hpp#L421],
 thus providing an incorrect size.


was (Author: edponce):
Well, the issue is that `string_view` is used to encapsulate binary data in 
certain parts and [`string_view.length()` is used to get the size but it stops 
at the first null 
byte|https://github.com/apache/arrow/blob/master/cpp/src/arrow/vendored/string_view.hpp#L421],
 thus providing an incorrect size.

> [C++] Mixed support for binary types in regex functions
> -------------------------------------------------------
>
>                 Key: ARROW-13879
>                 URL: https://issues.apache.org/jira/browse/ARROW-13879
>             Project: Apache Arrow
>          Issue Type: Bug
>          Components: C++
>            Reporter: Weston Pace
>            Assignee: Eduardo Ponce
>            Priority: Major
>              Labels: kernel, pull-request-available, types
>             Fix For: 6.0.0
>
>          Time Spent: 3h 50m
>  Remaining Estimate: 0h
>
> The functions count_substring, count_substring_regex, find_substring, and 
> find_substring_regex all accept binary types but the function extract_regex, 
> match_substring, match_substring_regex, match_like, starts_with, ends_with, 
> split_pattern, and split_pattern_regex do not.
> They should either all accept binary types or none should.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to