[GitHub] [nifi] timeabarna commented on a diff in pull request #7315: NIFI-3065 Per Process Group logging

["timeabarna (via GitHub)" <gi...@apache.org>]

timeabarna commented on code in PR #7315:
URL: https://github.com/apache/nifi/pull/7315#discussion_r1219197176


##########
nifi-api/src/main/java/org/apache/nifi/flow/VersionedProcessGroup.java:
##########
@@ -47,6 +47,9 @@ public class VersionedProcessGroup extends VersionedComponent {
     private Long defaultBackPressureObjectThreshold;
     private String defaultBackPressureDataSizeThreshold;
 
+    private Boolean logToOwnFile;

Review Comment:
   Thanks @markap14 for your review.
   For better User experience I've separated the enabling and configuration. There is a combolist with 'Yes' and 'No' options to turn dedicated logging and there is an input field with Process Group name as a default value for the log file name. I think this way the UI is more intuitive and more convenient than typing/deleting log file suffix.



-- 
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: issues-unsubscribe@nifi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org