[GitHub] [nifi] gresockj commented on a change in pull request #5450: NIFI-9292: Added getInputPortNames() and getOutputPortNames() to Stat…

[GitBox <gi...@apache.org>]

gresockj commented on a change in pull request #5450:
URL: https://github.com/apache/nifi/pull/5450#discussion_r730398202



##########
File path: nifi-stateless/nifi-stateless-api/src/main/java/org/apache/nifi/stateless/config/PropertiesFileEngineConfigurationParser.java
##########
@@ -119,6 +121,11 @@ public File getExtensionsDirectory() {
                 return extensionsDirectory;
             }
 
+            @Override
+            public Collection<File> getReadOnlyExtensionsDirectories() {
+                return Collections.emptyList();

Review comment:
       I understand that code outside of the Stateless framework could choose to implement a `StatelessEngineConfiguration` that provides this value, but perhaps it would be worth adding support for it in this class as well.  What do you think?  And if so, the documentation should also be updated.




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