Github user dbtsai commented on the issue:
https://github.com/apache/spark/pull/21495
I was on a family leave for couple weeks. Thank you all for helping out and
merging it.
The only change with this PR is that the welcome message will be printed
first, and then the Spark URL will be shown latter. It's a minor difference.
I had an offline discussion with @adriaanm in Scala community. To overcome
this issue,
he suggested we can override the entire `def process(settings: Settings):
Boolean` to put our initialization code. I have a working implementation of
this, but this will copy a lot of code from Scala to just get the printing
order right. If we decide to have a consistent printing order, I can submit a
separate PR for this.
He is open to working with us to add proper hook in Scala so we don't need
use hacks to initialize our code.
Thanks.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
