markap14 commented on code in PR #9292:
URL: https://github.com/apache/nifi/pull/9292#discussion_r1777113039
##########
nifi-extension-bundles/nifi-py4j-extension-bundle/nifi-py4j-bridge/src/main/java/org/apache/nifi/python/processor/FlowFileTransformProxy.java:
##########
@@ -40,22 +40,22 @@ public FlowFileTransformProxy(final String processorType,
final Supplier<PythonP
@Override
public void onTrigger(final ProcessContext context, final ProcessSession
session) throws ProcessException {
- FlowFile original = session.get();
- if (original == null) {
+ FlowFile transformed = session.get();
Review Comment:
It feels weird to me to call the FlowFile that we are pulling from an input
queue `transformed`. Perhaps we should name the variable simply `flowFile`? And
then we can call the clone just simply `clone` rather than `originalCloned`? I
think that would make the code a little easier to read personally.
--
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]
