exceptionfactory commented on code in PR #8943:
URL: https://github.com/apache/nifi/pull/8943#discussion_r1635760734


##########
nifi-extension-bundles/nifi-py4j-bundle/nifi-py4j-bridge/src/main/java/org/apache/nifi/python/processor/RecordTransformProxy.java:
##########
@@ -264,13 +265,16 @@ private void writeResult(final RecordTransformResult 
result, final Map<RecordGro
             final FlowFile destinationFlowFile = 
session.create(originalFlowFile);
 
             final RecordSetWriter writer;
+            OutputStream out = null;

Review Comment:
   Is there a reason for declaring this outside of the try-block instead of 
using try-with-resources? That should ensure the OutputStream is closed without 
the need for closeQuietly.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to