zanmato1984 commented on PR #43832: URL: https://github.com/apache/arrow/pull/43832#issuecomment-2485626030
> Some comments and questions below. Are we sure these are tested thoroughly enough? Just a quick answer to this specific question: yes the code changed are in a very common path that almost every swiss join case will run into, and my experience of developing/debugging this feature told me so too. So I'm positive that existing tests exercise them well. And thank you for the rest of the thorough comments. Will get to them later. -- 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]
