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 2020/08/15 06:32:02 UTC

[GitHub] [arrow] jorgecarleitao edited a comment on pull request #7878: ARROW-9617: [Rust] [DataFusion] Add length of string array

jorgecarleitao edited a comment on pull request #7878:
URL: https://github.com/apache/arrow/pull/7878#issuecomment-674357343


   There are now two distinct commits. The first one is the length for DataFusion as a UDF, the second one simplifies how we enumerate scalar functions that can be registered on the context.
   
   The main rational for the second commit is that the physical plan should not have to worry about how an `ExecutionContext` registers udfs. Instead, it should just enumerate which udfs are available, and let the `ExecutionContext` register them itself. We can revert this if we would prefer to place it on a second PR / not make the change.


----------------------------------------------------------------
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:
users@infra.apache.org