[SPARK-8611] [WebUI] Fix wrong REST UI Protocol on standalone cluster Web UI Page

[thegiive]

Just easily change the "spark://" to "http://"

[srowen]

OK to test

[srowen]

That looks like it could be right but can you confirm @andrewor14 ? this was added in 1390e56

[thegiive]

Hi Sean

What is the proper way to confirm with @andrewor14 ? By email?

[srowen]

No an @ mention is fine.

[thegiive]

Hi, is there anyone who can confirm this patch? @andrewor14 or someone else

[andrewor14]

Hi sorry for slipping I'll look at this today.

[andrewor14]

please add spaces before and after the =

[andrewor14]

@thegiive two things:

• This patch seems to include your other patch [SPARK-8656][WebUI] Fix the webUI and JSON API number is not synced #7038. Is this a mistake?
• Why do you think it's supposed to be HTTP? The restUri here refers to the Spark master identifier, e.g. users will submit their applications using:

bin/spark-submit --master spark://localhost:6066

Internally this uses REST / HTTP to submit it, but the external interface should use the spark:// prefix to signify that this is Spark standalone mode.

I believe this fix here is incorrect. Would you mind closing this patch?

[thegiive]

OK. It looks wired in first. But your comment looks reasonable.

BTW, I found this PR is close already.