contrib
#2191

Closed

Drill 7882 + Drill 7883 - Fix LGTM Alerts in /common and /contrib #2191

eevanwong wants to merge 8 commits into apache:master from eevanwong:LGTM

luocooong added code-cleanup

luocooong assigned

eevanwong 3 years ago

luocooong commented on 2021-03-24

eevanwong marked this pull request as draft 3 years ago

eevanwong force pushed from 5483399f to 712f3f99 3 years ago

luocooong closed this 3 years ago

luocooong reopened this 3 years ago

luocooong commented on 2021-04-10

cgivre requested changes on 2021-04-13

Drill 7882 + Drill 7883 - Fix LGTM Alerts in /common and /contrib

554eb53d

LGTM in contrib/ and common/ update

9d271a12

Update MaprDBJsonRecordReader.java

fb8fe6d7

Update HDF5BatchReader.java

24d7e416

Update JsonTableGroupScan.java

4baaca6a

Update XMLBatchReader.java

c7da6049

Update XMLBatchReader.java

286de5fd

Update NetworkFunctions.java

0071b911

eevanwong force pushed from 3a7cb78e to 0071b911 3 years ago

cgivre closed this 2 years ago

Reviewers

cgivre

luocooong

Assignees

eevanwong

Labels

code-cleanup

Milestone

No milestone