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

    https://github.com/apache/metron/pull/975#discussion_r180833783
  
    --- Diff: 
metron-platform/metron-solr/src/main/java/org/apache/metron/solr/dao/SolrDao.java
 ---
    @@ -72,12 +72,12 @@ public SolrDao() {
       @Override
       public void init(AccessConfig config) {
         if (config.getKerberosEnabled()) {
    -      HttpClientUtil.addConfigurer(new Krb5HttpClientConfigurer());
    +      enableKerberos();
    --- End diff --
    
    Thank you for the refactorings like this.  Even though small, they help 
with readability a bunch.  Bravo.


---

Reply via email to