Github user mmiklavc commented on a diff in the pull request:

    https://github.com/apache/metron/pull/853#discussion_r168802913
  
    --- Diff: 
metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/rest_commands.py
 ---
    @@ -179,6 +203,35 @@ def status_rest_application(self, env):
                 self.__params.metron_rest_port,
                 self.__params.metron_user)
     
    +    def create_hbase_tables(self):
    +        Logger.info("Creating HBase Tables")
    +        metron_service.create_hbase_table(self.__params,
    +                                          
self.__params.user_settings_hbase_table,
    +                                          
self.__params.user_settings_hbase_cf)
    +        Logger.info("Done creating HBase Tables")
    +        self.set_hbase_configured()
    +
    +    def set_hbase_acls(self):
    +        Logger.info("Setting HBase ACLs")
    +        if self.__params.security_enabled:
    +            kinit(self.__params.kinit_path_local,
    +                  self.__params.hbase_keytab_path,
    +                  self.__params.hbase_principal_name,
    +                  execute_user=self.__params.hbase_user)
    +
    +        cmd = "echo \"grant '{0}', 'RW', '{1}'\" | hbase shell -n"
    --- End diff --
    
    Do we check that the REST API has the hbase client also installed on the 
node? This would be needed for this to work for > 1 nodes.


---

Reply via email to