bkietz commented on PR #46180:
URL: https://github.com/apache/arrow/pull/46180#issuecomment-2847613487

   > How do we feel about the wrappers in gtest_util still using the old name 
(ArrayFromJSON, etc) without the String suffix?
   
   I think changing all of those at once would represent massive churn in the 
codebase and it'd be better to have a testing-only alias for now. As a follow 
up, a PR which only removes that alias could be added to 
[blame.ignoreRevsFile](https://git-scm.com/docs/git-blame#Documentation/git-blame.txt---ignore-revs-fileltfilegt)
 to avoid clobbering git blame


-- 
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-unsubscr...@arrow.apache.org

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

Reply via email to