Quantcast

Why separate SessionStateBuilder? (it's BaseSessionStateBuilder)

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Why separate SessionStateBuilder? (it's BaseSessionStateBuilder)

Jacek Laskowski
Hi,

While reviewing SessionStateBuilder [1] I came to conclusion that it's
a kind of a leftover and should really be removed since it's
BaseSessionStateBuilder [2].

What I'm proposing is to remove SessionStateBuilder followed by
renaming BaseSessionStateBuilder to SessionStateBuilder.

That will make things simpler/clearer.

[1] https://github.com/apache/spark/blob/master/sql/core/src/main/scala/org/apache/spark/sql/internal/SessionState.scala#L116
[2] https://github.com/apache/spark/blob/master/sql/core/src/main/scala/org/apache/spark/sql/internal/BaseSessionStateBuilder.scala#L54

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

---------------------------------------------------------------------
To unsubscribe e-mail: [hidden email]

Loading...