eevanwong commented on pull request #2187: URL: https://github.com/apache/drill/pull/2187#issuecomment-801261325
@vvysotskyi yeah for sure. A big thing is that we want to make sure that what we're changing/surpressing isnt too major. So with each alert (or file w/ alerts) I have a small justification of why I surpressed/changed it. It'd be really cluttered if we put them all on 3 big prs. Would you prefer it if we had smaller prs, or just do as many as possible? ---------------------------------------------------------------- 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]
