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

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2017/12/19 05:14:53 UTC

[GitHub] nkurihar commented on issue #820: Added setting for anonymous user role

nkurihar commented on issue #820: Added setting for anonymous user role
URL: https://github.com/apache/incubator-pulsar/pull/820#issuecomment-341319570
 
 
   I also think:
   * `authenticationEnabled` is redundant any more
   * removal of `authenticationEnabled` should be done by another PR, and its milestone should be 2.0.0 (since it will break the configuration file compatibility)
   
   The change in this PR looks good to me.
   
   @merlimat 
   Can we merge this PR for 1.X.0-incubating?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services