jfrazee commented on a change in pull request #4613:
URL: https://github.com/apache/nifi/pull/4613#discussion_r510327902
##########
File path:
nifi-commons/nifi-properties/src/main/java/org/apache/nifi/util/NiFiProperties.java
##########
@@ -1335,6 +1335,13 @@ public boolean isZooKeeperClientSecure() {
return Boolean.parseBoolean(clientSecure);
}
+ public boolean isZooKeeperTlsConfigurationPresent() {
Review comment:
Just to sanity check. This doesn't
include`NiFiProperties.ZOOKEEPER_CLIENT_SECURE` since it's possible to have
cluster ZK security without state management ZK security and vice versa?
##########
File path:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/controller/state/manager/StandardStateManagerProvider.java
##########
@@ -76,6 +77,8 @@ private StandardStateManagerProvider(final StateProvider
localStateProvider, fin
this.clusterStateProvider = clusterStateProvider;
}
+ public static Map<String, String> tlsPropertyMappings;
Review comment:
I don't think this gets used anywhere.
```suggestion
```
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
