[SPARK-5733] Error Link in Pagination of HistroyPage when showing Incomplete Applications

[marsishandsome]

The links in pagination of HistroyPage is wrong when showing Incomplete Applications.

If "2" is click on the following page "http://history-server:18080/?page=1&showIncomplete=true", it will go to "http://history-server:18080/?page=2" instead of "http://history-server:18080/?page=2&showIncomplete=true".

[AmplabJenkins]

Can one of the admins verify this patch?

[srowen]

ok to test

[SparkQA]

Test build #27281 has started for PR 4523 at commit 9d7b593.

• This patch merges cleanly.

[srowen]

Just from reading this page, LGTM. Seems clear that the utility function should be used everywhere to make the page link. Ideally @sarutak can confirm (see #3194)

[SparkQA]

Test build #27281 has finished for PR 4523 at commit 9d7b593.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds the following public classes (experimental):
  • case class TaskCommitDenied(
  • class CommitDeniedException(
  • class OutputCommitCoordinatorActor(outputCommitCoordinator: OutputCommitCoordinator)

[AmplabJenkins]

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27281/
Test PASSed.

[sarutak]

I applied this changes and confirmed. It's LGTM, thanks @marsishandsome !