martinzink commented on code in PR #2246:
URL: https://github.com/apache/nifi-minifi-cpp/pull/2246#discussion_r3881926074


##########
minifi_rust/extensions/minifi_rs_playground/src/processors/put_file/processor_definition.rs:
##########


Review Comment:
   It was done so we can concat properties to form composite processors and 
refer to the properties of the parts. (we cannot concat in const context 
without it being fn)
   Its not used here but I've used it in an upcoming PR (the framework related 
changes have been extracted to this PR)
   
   Since im not sure I'll extract this change into a separate PR (or include it 
in the upcoming composite processor's PR) so we can discuss it there and weigh 
the pros and cons. 



-- 
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