You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/04/13 08:10:44 UTC

[GitHub] [arrow-rs] HaoYang670 commented on pull request #1543: Do not merge! Rename the `substring` function and mark it as `unsafe`

HaoYang670 commented on PR #1543:
URL: https://github.com/apache/arrow-rs/pull/1543#issuecomment-1097689280

   Strongly agree with you @alamb. But I have some different opintion with you on the renaming after considering more about the back compatibility. Please look at the comment https://github.com/apache/arrow-rs/issues/1531#issuecomment-1097613212
   
   We'd better reach agreement on the renaming before doing the following tasks 


-- 
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.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

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