Hi,

Analyzer.resolver returns a Resolver per CatalystConf setting [1] that
seems a duplicate of CatalystConf.resolver.

Unless I'm mistaken, the code could get simpler and be as follows:

def resolver: Resolver = conf.resolver

Would you agree? I'm ready with a PR if so (plus some other typo fixes).

[1] 
https://github.com/apache/spark/blob/master/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala#L67-L73

[2] 
https://github.com/apache/spark/blob/master/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/CatalystConf.scala#L43-L45

Pozdrawiam,
Jacek Laskowski
----
https://medium.com/@jaceklaskowski/
Mastering Apache Spark 2.0 http://bit.ly/mastering-apache-spark
Follow me at https://twitter.com/jaceklaskowski

---------------------------------------------------------------------
To unsubscribe e-mail: dev-unsubscr...@spark.apache.org

Reply via email to