milevin commented on PR #6906:
URL: https://github.com/apache/arrow-rs/pull/6906#issuecomment-2557792334

   > > The approach I suggested doesn't rely on these Datafusion casts and 
doesn't increase binary size. It's a win-win.
   > 
   > Unfortunately it is incorrect to treat intervals in this way - [#6906 
(comment)](https://github.com/apache/arrow-rs/pull/6906#discussion_r1894338923)
   > 
   > I will try to find some time to re-review this PR over the next few days
   
   Thanks a lot to both of you for the feedback!
   
   I'll wait for @tustvold's additional review in the next few days and proceed 
based on the combination of recommendations.


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