kennknowles commented on pull request #12568: URL: https://github.com/apache/beam/pull/12568#issuecomment-682511813
I was just tagging folks who I think have the best knowledge. It seems between these PRs we already have a number of people who know generally what needs to be done. Judging by the other PR, maybe not everything here is mandatory for the upgrade? Then it could be quite a lot smaller, or broken into more commits. I have no problem with the change from `compile` to `implementation`, etc, but making sure we got everything and making sure nothing went wrong makes it a bit more work to review. ---------------------------------------------------------------- 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: us...@infra.apache.org
