metron-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From merrimanr <...@git.apache.org>
Subject [GitHub] metron issue #891: METRON-1282 creating a new topic using the rest end point...
Date Tue, 09 Jan 2018 17:41:54 GMT
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.


---

Mime
View raw message