rok edited a comment on pull request #11990:
URL: https://github.com/apache/arrow/pull/11990#issuecomment-999515836


   > Updated to make YearMonthDay work more closely to the other Unary funcs. 
This doesn't build atm because (as far as I can tell) passing a StructArray to 
the `UnaryTemporalFactory` doesn't resolve to existing specializations. Need to 
study the factory machinery a little more, but I'm guessing something needs to 
be changed with the factories themselves to allow this
   
   I think this is also the reason `ISOCalendar` is implemented the way is was.
   
   @lidavidm can unary kernels return StructArray? Also, can you approve the CI 
workflow? I don't have permissions to.


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