Skip to content

SPARK-4579 [WEBUI] Scheduling Delay appears negative - #4796

Closed
srowen wants to merge 1 commit into
apache:masterfrom
srowen:SPARK-4579
Closed

SPARK-4579 [WEBUI] Scheduling Delay appears negative#4796
srowen wants to merge 1 commit into
apache:masterfrom
srowen:SPARK-4579

Conversation

@srowen

@srowensrowen commented Feb 26, 2015

Copy link
Copy Markdown
Member

Ensure scheduler delay handles unfinished task case, and ensure delay is never negative even due to rounding

@SparkQA

SparkQA commented Feb 26, 2015

Copy link
Copy Markdown

Test build #28024 has started for PR 4796 at commit ad6713c.

  • This patch merges cleanly.

@SparkQA

SparkQA commented Feb 26, 2015

Copy link
Copy Markdown

Test build #28024 has finished for PR 4796 at commit ad6713c.

  • This patch fails Python style tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@AmplabJenkins

AmplabJenkins commented Feb 26, 2015

Copy link
Copy Markdown

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

@kayousterhout

kayousterhout commented Feb 26, 2015

Copy link
Copy Markdown
Contributor

LGTM Thanks @srowen !

@pwendell

pwendell commented Feb 26, 2015

Copy link
Copy Markdown
Contributor

Jenkins, test this please.

@pwendell

pwendell commented Feb 26, 2015

Copy link
Copy Markdown
Contributor

Yeah this would be great to pull into an RC... a very confusing presentation in the UI right now!

@SparkQA

SparkQA commented Feb 26, 2015

Copy link
Copy Markdown

Test build #28027 has started for PR 4796 at commit ad6713c.

  • This patch merges cleanly.

@SparkQA

SparkQA commented Feb 27, 2015

Copy link
Copy Markdown

Test build #28027 has finished for PR 4796 at commit ad6713c.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@AmplabJenkins

AmplabJenkins commented Feb 27, 2015

Copy link
Copy Markdown

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

@andrewor14

andrewor14 commented Feb 27, 2015

Copy link
Copy Markdown
Contributor

Ok I've pulled this into master 1.3 and 1.2 thanks.

asfgit pushed a commit that referenced this pull request Feb 27, 2015
@srowenAndrew Or
SPARK-4579[WEBUI] Scheduling Delay appears negative
d0bf938
Ensure scheduler delay handles unfinished task case, and ensure delay is never negative even due to rounding
Author: Sean Owen <sowen@cloudera.com>
Closes#4796 from srowen/SPARK-4579 and squashes the following commits:
ad6713c [Sean Owen] Ensure scheduler delay handles unfinished task case, and ensure delay is never negative even due to rounding
asfgit pushed a commit that referenced this pull request Feb 27, 2015
@srowenAndrew Or
SPARK-4579[WEBUI] Scheduling Delay appears negative
b83a93e
Ensure scheduler delay handles unfinished task case, and ensure delay is never negative even due to rounding
Author: Sean Owen <sowen@cloudera.com>
Closes#4796 from srowen/SPARK-4579 and squashes the following commits:
ad6713c [Sean Owen] Ensure scheduler delay handles unfinished task case, and ensure delay is never negative even due to rounding
(cherry picked from commit fbc4694)
Signed-off-by: Andrew Or <andrew@databricks.com>
@srowen
srowen deleted the SPARK-4579 branch February 27, 2015 13:28
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@srowen@SparkQA@AmplabJenkins@kayousterhout@pwendell@andrewor14