waynexia commented on PR #5836: URL: https://github.com/apache/arrow-rs/pull/5836#issuecomment-2149216144
>The one exception to this are the symmetric hash join, but given the amount of interval-specific logic in DF for these I am inclined to think the issue likely does not lie in arrow-rs. The fix commit is https://github.com/apache/datafusion/pull/10765/commits/fc9a86bd66659e0f0d87f65a18b3c150028e4fce. I'm not sure if `IntervalDayTime::ONE` should be "1 day 1 milli" or "0 day 1 milli" 🤔 But this should not be a big problem as we can choose not to use that const if its value is not expected. Only concern about it may bring some misunderstanding. -- 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]
