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

    https://github.com/apache/metron/pull/759#discussion_r139013691
  
    --- Diff: 
metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/indexing_commands.py
 ---
    @@ -46,6 +46,7 @@ def __init__(self, params):
             self.__acl_configured = 
os.path.isfile(self.__params.indexing_acl_configured_flag_file)
             self.__hbase_configured = 
os.path.isfile(self.__params.indexing_hbase_configured_flag_file)
             self.__hbase_acl_configured = 
os.path.isfile(self.__params.indexing_hbase_acl_configured_flag_file)
    +        self.__hdfs_perm_configured = 
os.path.isfile(self.__params.indexing_hdfs_perm_configured_flag_file)
    --- End diff --
    
    This is another issue that I found.  I am not sure what the impact is, but 
seems like we would just always set the HDFS perms.


---

Reply via email to