apilloud commented on pull request #12247: URL: https://github.com/apache/beam/pull/12247#issuecomment-657905365
I would be fine with deleting anything that is obviously broken and cleanup, but this isn't dead code. I know of at least one customer use case where they are disabling `BeamZetaSqlCalcRel` so they can avoid JNI. We also have plans to turn `BeamCalcRel` back on for simple operators to enable better performance. Also looks like you are doing `git pull` with the default config, which is creating merge commits on your branch. I believe the preferred method is `git pull --rebase` or `git rebase upstream/master`. (There is a way to make that the default, I don't remember of the top of my head.) ---------------------------------------------------------------- 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]
