You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@metron.apache.org by nickwallen <gi...@git.apache.org> on 2017/09/20 20:43:45 UTC

[GitHub] metron pull request #726: METRON-1145: Profiler mpack does not create kafka ...

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?


---