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

Posted to dev@metron.apache.org by merrimanr <gi...@git.apache.org> on 2018/01/09 17:41:54 UTC

[GitHub] metron issue #891: METRON-1282 creating a new topic using the rest end point...

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.


---