[ https://issues.apache.org/jira/browse/DRILL-8268?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17572574#comment-17572574 ]
ASF GitHub Bot commented on DRILL-8268: --------------------------------------- vvysotskyi commented on code in PR #2610: URL: https://github.com/apache/drill/pull/2610#discussion_r932494153 ########## exec/java-exec/src/main/java/org/apache/drill/exec/store/easy/sequencefile/SequenceFileBatchReader.java: ########## @@ -154,6 +153,19 @@ public boolean next() { @Override public void close() { - AutoCloseables.closeSilently(reader); + try { + // Hadoop 2 compat: {@link org.apache.hadoop.mapred.RecordReader} does not + // support AutoCloseable and must be closed manually. + if (reader != null) { + reader.close(); + reader = null; + } + } catch (IOException e) { + throw UserException Review Comment: `AutoCloseables.closeSilently` doesn't throw any errors. ########## exec/jdbc-all/pom.xml: ########## @@ -1230,7 +1230,7 @@ <profile> <id>hadoop-2</id> <properties> - <jdbc-all-jar.maxsize>50400000</jdbc-all-jar.maxsize> + <jdbc-all-jar.maxsize>52800000</jdbc-all-jar.maxsize> Review Comment: Can we exclude some unnecessary dependencies from JDBC Driver to avoid its inflation? > Fix Hadoop 2 and Netty lib exclusions, REST mem limiter disabled by default > --------------------------------------------------------------------------- > > Key: DRILL-8268 > URL: https://issues.apache.org/jira/browse/DRILL-8268 > Project: Apache Drill > Issue Type: Bug > Affects Versions: 1.20.1 > Reporter: James Turton > Assignee: James Turton > Priority: Major > Fix For: 1.20.2 > > > # New exclusions of reload4j, slf4j-reload4j are required in the Hadoop 2 > profile, probably due to the upgrade of Hadoop from 2.10.1 to 2.10.2. > # We remove the netty-all metapackage which entered the dependency tree with > the change introducing the Netty bom bringing many uneeded libs with it. > # The heap memory usage limiting logic in the REST server becomes disabled > by default since REST query results are streamed these days. This change aims > to let the Java GC now do its job without interference and if that results in > OOM under a constant load then there is good evidence for a heap leak which > must be tracked down and completely resolved anyway, not mitigated or "swept > under a rug". -- This message was sent by Atlassian Jira (v8.20.10#820010)