Evan W.

@vvysotskyi Honestly there are a lot of stuff there that I'd be down to fix. @mkusnir has shown interest as well. Would it be alright if we made a few...

@cgivre Hey I've made the JIRA https://issues.apache.org/jira/projects/DRILL/issues/DRILL-7878?filter=updatedrecently, is there anything should I add to them?

@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...

As a side note: I had not touched Drillbuf.java, or Drillfilesystem.java, but upon making this I found them. I can remove them if they're unnecessary.

Errors in CI involve drillbuf.java, DrillFileSystem.java. Fixed

Hey, yeah there are a lot of suppressions. The next step for me is to fully understand all the ??? ones, then determining if I can fix. I just wanted...

Hey, could I get someones input on this? This is quite a small thing, but still something I want to make sure.  This is in JsonTableGroupScan.java, line 519. The...

The most common issues that I suppressed involved null values and potential NPE and override hashcode and equals. The thing with the null values is that, most times in the...

1. According to their website, LGTM takes the codebase, "generates a database to represent the codebase", then uses codeQL to run queries on the database to generate alerts. They use...

1. Yes there are 17 alerts in total caused by this issue.  2. I'm not sure if the null errors are related, the main issue on the alert was...