ozankabak commented on PR #5603:
URL: 
https://github.com/apache/arrow-datafusion/pull/5603#issuecomment-1474432417

   @alamb, thanks for reviewing. As you correctly guessed, this is for things 
like determining window boundaries, interval calculations for pruning etc. It 
will not be used on raw data when executing queries (we will use arrow kernels 
there).
   
   @berkaysynnada is working on the arrow kernel parts and we will follow on 
with another PR when it is ready. It will include both end to end tests (i.e. 
running queries) and the consistency tests you suggested.
   
   I cleaned up the overly verbose interval construction boilerplate, so this 
is good to go in terms of the feedback so far. We will follow up with the other 
PR early next week after this merges.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to