Skip to content

Conversation

@ajbozarth
Copy link
Member

What changes were proposed in this pull request?

I fixed a misassigned var, numCompletedTasks was assigned to numSkippedTasks in the convertJobData method

How was this patch tested?

dev/run-tests

@SparkQA
Copy link

SparkQA commented Jul 12, 2016

Test build #62120 has finished for PR 14141 at commit 7ac60de.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@ajbozarth
Copy link
Member Author

apparently dev/run-tests didn't actually finish, I'll fix the test failures

@SparkQA
Copy link

SparkQA commented Jul 12, 2016

Test build #62127 has finished for PR 14141 at commit 5122a1a.

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

@srowen
Copy link
Member

srowen commented Jul 12, 2016

Oops, great catch

@srowen
Copy link
Member

srowen commented Jul 13, 2016

Merged to master/2.0/1.6

asfgit pushed a commit that referenced this pull request Jul 13, 2016
…signed to numSkippedTasks

## What changes were proposed in this pull request?

I fixed a misassigned var,  numCompletedTasks was assigned to numSkippedTasks in the convertJobData method

## How was this patch tested?

dev/run-tests

Author: Alex Bozarth <[email protected]>

Closes #14141 from ajbozarth/spark16375.

(cherry picked from commit f156136)
Signed-off-by: Sean Owen <[email protected]>
@asfgit asfgit closed this in f156136 Jul 13, 2016
asfgit pushed a commit that referenced this pull request Jul 13, 2016
…signed to numSkippedTasks

## What changes were proposed in this pull request?

I fixed a misassigned var,  numCompletedTasks was assigned to numSkippedTasks in the convertJobData method

## How was this patch tested?

dev/run-tests

Author: Alex Bozarth <[email protected]>

Closes #14141 from ajbozarth/spark16375.

(cherry picked from commit f156136)
Signed-off-by: Sean Owen <[email protected]>
zzcclp pushed a commit to zzcclp/spark that referenced this pull request Jul 13, 2016
…signed to numSkippedTasks

## What changes were proposed in this pull request?

I fixed a misassigned var,  numCompletedTasks was assigned to numSkippedTasks in the convertJobData method

## How was this patch tested?

dev/run-tests

Author: Alex Bozarth <[email protected]>

Closes apache#14141 from ajbozarth/spark16375.

(cherry picked from commit f156136)
Signed-off-by: Sean Owen <[email protected]>
(cherry picked from commit fb09336)
@ajbozarth ajbozarth deleted the spark16375 branch July 13, 2016 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants