wesm edited a comment on pull request #7593:
URL: https://github.com/apache/arrow/pull/7593#issuecomment-652053744


   I also propose to rename the function from "contains_exact" to 
"utf8_contains". ~~I'm pushing that change here shortly~~. Or should we call it 
"binary_contains" / "string_contains" since it will work with any base binary 
types? I'll hold off on making any changes until there is consensus about the 
name. I think it would be useful to put string functions in a 
"pseudo-namespace" so they appear next to each other when sorted 
alphabetically. 
   
   Ideas:
   
   - string_contains
   - string_lower_utf8
   - string_lower_ascii
   - ...
   
   @maartenbreddels @pitrou any opinions about this?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to