Github user kayousterhout commented on the pull request:
https://github.com/apache/spark/pull/3009#issuecomment-63855330
Also, I thought more about having this in 1.2, and I'm -0.5 on putting this
in 1.2. Given all of the subtleties you ended up running into with this, Josh,
I don't think it's a good idea to put it in the release at the last minute
without giving folks plenty of time to test it out. Based on my (admittedly
limited!) understanding of most Spark users, the UI page is one of the first
thing a user will look at, and I don't think pushing a major change to what the
user first sees when he or she interacts with Spark this late in the release
cycle is a good idea. If I were a Spark user and had successfully tried the
preview release Patrick had posted, and then later found that the final 1.2
release changed the landing page for the UI in a buggy / unintuitive / <insert
unexpected bug here>, I think I'd be annoyed and question the Spark QA process.
I'm certainly willing to be overruled if others are less concerned about
the points I mentioned.
---
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
