[
https://issues.apache.org/jira/browse/DRILL-3927?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14957724#comment-14957724
]
ASF GitHub Bot commented on DRILL-3927:
---------------------------------------
Github user cwestin commented on a diff in the pull request:
https://github.com/apache/drill/pull/199#discussion_r42050675
--- Diff:
exec/jdbc/src/main/java/org/apache/drill/jdbc/impl/DrillConnectionImpl.java ---
@@ -369,24 +370,33 @@ AvaticaFactory getFactory() {
return factory;
}
+ private static void closeOrWarn(final AutoCloseable autoCloseable, final
String message, final Logger logger) {
--- End diff --
Sounds good, but let's leave that until DrillAutoCloseables and
AutoCloseables are merged, along with the change you suggested earlier/above.
> Use OutOfMemoryException in more places
> ---------------------------------------
>
> Key: DRILL-3927
> URL: https://issues.apache.org/jira/browse/DRILL-3927
> Project: Apache Drill
> Issue Type: Bug
> Components: Execution - Flow
> Reporter: Chris Westin
> Assignee: Chris Westin
>
> The new allocator uses OutOfMemoryException in better ways; some additional
> exception handling sites need to catch and handle this exception in
> preparation for the new allocator.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
