[GitHub] drill pull request #1023: DRILL-5922 Fixed Child Allocator Leak. DRILL-5926 ...

[paul-rogers]

Github user paul-rogers commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1023#discussion_r148997637
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/work/user/PlanSplitter.java 
---
    @@ -79,6 +80,15 @@ public QueryPlanFragments planFragments(DrillbitContext 
dContext, QueryId queryI
           responseBuilder.setStatus(QueryState.FAILED);
           responseBuilder.setError(error);
         }
    +
    +    try {
    +      queryContext.close();
    +    } catch (Exception e) {
    +      final String message = String.format("Error closing QueryContext 
when getting plan fragments for query %s",
    +        QueryIdHelper.getQueryId(queryId));
    +      logger.error(message, e);
    --- End diff --
    
    See above re log message formatting.


---