Github user steveloughran commented on the issue:

    https://github.com/apache/spark/pull/15594
  
    I'm not spark committer so can't review it well enough to get in; I was 
just watching it out of concern for the word "kerberos". How about you ask on 
the spark developer list for any volunteers to review.
    
    Looking at the code, I'd drop using reflection for logging. You can just 
create an SLF4J log with the name "org.apache.hive.hiveserver2" (or whatever 
the full path is) and end up with the same log name as the parent class. As for 
the setting of the hive option, I do think it's an ugly pain which should 
really be addressed by making hive extensible —until then, what choice to do 
you have.



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to