jorisvandenbossche commented on pull request #12215: URL: https://github.com/apache/arrow/pull/12215#issuecomment-1047537483
> Added the test and removed divide checked (seemed redundant since we're only dividing with integers). I would keep `divide_checked`. You could give the same argument for divide_checked being redundant for integer types, but we still allow you to use it. It could be annoying that, when using divide_checked, you need to switch to divide, only if your input type is duration. -- 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]
