ianmcook commented on a change in pull request #9745:
URL: https://github.com/apache/arrow/pull/9745#discussion_r597976468
##########
File path: r/R/expression.R
##########
@@ -181,6 +181,20 @@ find_array_refs <- function(x) {
unlist(out)
}
+get_field_name <- function(x, msg = NULL) {
Review comment:
I should probably rename this. It takes a single `array_expression` as
input. If the expression is just a bare column reference, it returns its name
deparsed as a character string. If the expression is something more than just a
bare column reference, it errors. This function will change or go away when I
implement support for sorting by expressions.
--
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
