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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
