Skip to content

Conversation

@zsxwing
Copy link
Member

@zsxwing zsxwing commented Mar 4, 2017

What changes were proposed in this pull request?

"DataFrameCallbackSuite.execute callback functions when a DataFrame action failed" sets the log level to "fatal" but doesn't recover it. Hence, tests running after it won't output any logs except fatal logs.

This PR uses testQuietly instead to avoid changing the log level.

How was this patch tested?

Jenkins

@zsxwing
Copy link
Member Author

zsxwing commented Mar 4, 2017

cc @cloud-fan @yhuai

@cloud-fan
Copy link
Contributor

LGTM

@SparkQA
Copy link

SparkQA commented Mar 4, 2017

Test build #73879 has finished for PR 17156 at commit 2db73ae.

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

@gatorsmile
Copy link
Member

Thanks! Merging to master.

@asfgit asfgit closed this in fbc4058 Mar 4, 2017
@yhuai
Copy link
Contributor

yhuai commented Mar 4, 2017

Let's also merge this to branch-2.1.

asfgit pushed a commit that referenced this pull request Mar 4, 2017
…esn't recover the log level

## What changes were proposed in this pull request?

"DataFrameCallbackSuite.execute callback functions when a DataFrame action failed" sets the log level to "fatal" but doesn't recover it. Hence, tests running after it won't output any logs except fatal logs.

This PR uses `testQuietly` instead to avoid changing the log level.

## How was this patch tested?

Jenkins

Author: Shixiong Zhu <[email protected]>

Closes #17156 from zsxwing/SPARK-19816.

(cherry picked from commit fbc4058)
Signed-off-by: Yin Huai <[email protected]>
@yhuai
Copy link
Contributor

yhuai commented Mar 4, 2017

merged to branch-2.1

@zsxwing zsxwing deleted the SPARK-19816 branch March 4, 2017 06:40
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.

5 participants