jnturton commented on PR #2671: URL: https://github.com/apache/drill/pull/2671#issuecomment-1269291191
> LGTM +1. I will say I'm a little nervous about merging an auto-generated PR w/o manual testing. @vvysotskyi Do you have any objections? I think that a very large number of our unit tests indirectly exercise protobuf-java making it safe enough to merge based on a CI run in my opinion. I'll also be doing manual tests later today and I can rebase to pull this is in if its merged, and keep an eye out for protobuf problems. -- 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: dev-unsubscr...@drill.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
