cgivre commented on PR #2631: URL: https://github.com/apache/drill/pull/2631#issuecomment-1236334438
I'm fine with merging as is, but there are a bunch of LGTM comments in the code to ignore false positives. Should we remove those here as well, or separate PR? -- 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