vertexclique edited a comment on pull request #8598: URL: https://github.com/apache/arrow/pull/8598#issuecomment-723470048
I don't think that's needed at all. Optimizations were extremely premature. We should have thought about the future beforehand. I think we are prolonging the discussion without any good point in it. Performance can improve later when we have correct code first. `+` This code is already improved a lot. I find all the comments kind of blocking after the effort I put into it. So ping me when you want to merge this. We need these changes anyway to run on other platforms. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
