Skip to content

Conversation

@ala
Copy link
Contributor

@ala ala commented May 10, 2017

What changes were proposed in this pull request?

b486ffc left behind references to "number of generated rows" metrics, that should have been removed.

How was this patch tested?

Existing unit tests.

@ala
Copy link
Contributor Author

ala commented May 10, 2017

@hvanhovell

@SparkQA
Copy link

SparkQA commented May 10, 2017

Test build #76749 has finished for PR 17939 at commit ffa8b80.

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

@hvanhovell
Copy link
Contributor

LGTM - merging to master/2.2. Thanks!

asfgit pushed a commit that referenced this pull request May 10, 2017
## What changes were proposed in this pull request?

b486ffc left behind references to "number of generated rows" metrics, that should have been removed.

## How was this patch tested?

Existing unit tests.

Author: Ala Luszczak <[email protected]>

Closes #17939 from ala/SPARK-19447-fix.

(cherry picked from commit 5c2c4dc)
Signed-off-by: Herman van Hovell <[email protected]>
@asfgit asfgit closed this in 5c2c4dc May 10, 2017
liyichao pushed a commit to liyichao/spark that referenced this pull request May 24, 2017
## What changes were proposed in this pull request?

apache@b486ffc left behind references to "number of generated rows" metrics, that should have been removed.

## How was this patch tested?

Existing unit tests.

Author: Ala Luszczak <[email protected]>

Closes apache#17939 from ala/SPARK-19447-fix.
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