Github user merrimanr commented on the issue:

    https://github.com/apache/metron/pull/891
  
    @MohanDV this is a good start and will be a nice feature to have.  I have a 
couple suggestions.  
    
    First, can we move this to it's own function/endpoint?  I think this would 
be useful for assigning permissions to existing topics as well.  
    
    Second, I think we need some kind of check that Kerberos is enabled before 
we try to permission a topic for a user.  This would be similar to what's done 
in the 
[KafkaConfig](https://github.com/apache/metron/blob/master/metron-interface/metron-rest/src/main/java/org/apache/metron/rest/config/KafkaConfig.java)
 class on line 111.


---

Reply via email to