Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/726#discussion_r140087395 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/profiler_master.py --- @@ -59,6 +60,9 @@ def configure(self, env, upgrade_type=None, config_dir=None): commands.create_hbase_tables() if params.security_enabled and not commands.is_hbase_acl_configured(): commands.set_hbase_acls() --- End diff -- Is there a reason that we do not call `commands.set_hbase_acl_configured()` here?
---