Skip to content

Conversation

@sarutak
Copy link
Member

@sarutak sarutak commented Sep 21, 2020

What changes were proposed in this pull request?

This PR reverts SPARK-32886 (#29757) for branch-2.4.
That change needs appBasePath in webui.js but it's absent for branch-2.4.

@sarutak
Copy link
Member Author

sarutak commented Sep 21, 2020

cc: @srowen @Dooyoung-Hwang

@SparkQA
Copy link

SparkQA commented Sep 21, 2020

Test build #128944 has finished for PR 29825 at commit c6705a8.

  • This patch fails build dependency tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@sarutak
Copy link
Member Author

sarutak commented Sep 21, 2020

retest this please.

@SparkQA
Copy link

SparkQA commented Sep 21, 2020

Test build #128945 has finished for PR 29825 at commit c6705a8.

  • This patch fails build dependency tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@srowen
Copy link
Member

srowen commented Sep 21, 2020

Ah OK, yes let's revert. Feel free to open another PR for branch-2.4 if you're up for it.

The test failure is not related, and another instance of a corrupt Maven dir on Jenkins. @shaneknapp can that be deleted?

So, I'm pretty OK to revert this right away as it clearly passed before the original change.

asfgit pushed a commit that referenced this pull request Sep 21, 2020
### What changes were proposed in this pull request?

This PR reverts SPARK-32886 (#29757) for branch-2.4.
That change needs `appBasePath` in `webui.js` but it's absent for `branch-2.4`.

Closes #29825 from sarutak/hotfix-for-SPARK-32886-2.4.

Authored-by: Kousuke Saruta <[email protected]>
Signed-off-by: Kousuke Saruta <[email protected]>
@sarutak
Copy link
Member Author

sarutak commented Sep 21, 2020

@srowen Thanks. I've reverted it.

@sarutak sarutak closed this Sep 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants