westonpace commented on PR #13880: URL: https://github.com/apache/arrow/pull/13880#issuecomment-1239665810
> Given the above, I see several options we have: Let's do 1 for now. I think this PR is probably ready to merge and it makes sense to chase those things down as follow-ups. Otherwise it might be too much to keep track of. Long term it sounds like 3 would be simpler if we could show it worked well. That being said, don't we only need to worry about collisions within the on field's tolerance? Or do we have to worry about a collision anywhere in the dataset? -- 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]
