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 2021/05/06 18:28:17 UTC

[GitHub] [arrow-datafusion] alamb commented on issue #251: Implement Postgres compatible `now()` function

alamb commented on issue #251:
URL: https://github.com/apache/arrow-datafusion/issues/251#issuecomment-833759034


   Hi @msathis  ! Thanks for giving this one a try. 
   
   One thing you might try is  just commenting out the `arg_types.is_empty()` check. The comments don't sound right to me (as a function always returns 1 row....) so maybe that check is now outdated. It may be that if no `Array` batch is passed to the implementation it doesn't know how many rows to output 🤔  
   


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