[GitHub] [arrow-datafusion] alamb commented on pull request #288: [Datafusion] NOW() function support

2021-05-14 Thread GitBox
alamb commented on pull request #288: URL: https://github.com/apache/arrow-datafusion/pull/288#issuecomment-841408299 I filed https://github.com/apache/arrow-datafusion/issues/340 to track possible future work on stateful functions -- This is an automated message from the Apache Git Serv

[GitHub] [arrow-datafusion] alamb commented on pull request #288: [Datafusion] NOW() function support

2021-05-13 Thread GitBox
alamb commented on pull request #288: URL: https://github.com/apache/arrow-datafusion/pull/288#issuecomment-840763111 @jorgecarleitao Also, I am not convinced about how valuable a general purpose `StatefulFunction` will be (though of course it depends on a proposal that is not yet written

[GitHub] [arrow-datafusion] alamb commented on pull request #288: [Datafusion] NOW() function support

2021-05-13 Thread GitBox
alamb commented on pull request #288: URL: https://github.com/apache/arrow-datafusion/pull/288#issuecomment-840759475 @jorgecarleitao I would like `now()` to be added to datafusion in the next few weeks -- my usecase is that I want to be able to write queries with predicates like "in the

[GitHub] [arrow-datafusion] alamb commented on pull request #288: [Datafusion] NOW() function support

2021-05-13 Thread GitBox
alamb commented on pull request #288: URL: https://github.com/apache/arrow-datafusion/pull/288#issuecomment-840491188 I am not opposed to the idea of `StatefulFunction` but it seems like a large overkill for the now() function. Here is an alternate proposal (in https://github.com/ap

[GitHub] [arrow-datafusion] alamb commented on pull request #288: [Datafusion] NOW() function support

2021-05-12 Thread GitBox
alamb commented on pull request #288: URL: https://github.com/apache/arrow-datafusion/pull/288#issuecomment-840101186 > The stateful function seems needed at some point. I am not sure a stateful function would be needed here, at this moment, and might go a bit beyond the scope of this PR/f