Skip to content

Conversation

@depend
Copy link
Contributor

@depend depend commented May 5, 2016

What changes were proposed in this pull request?

set log level to debug when check shouldRollover

How was this patch tested?

It's tested manually.

@srowen
Copy link
Member

srowen commented May 5, 2016

I tend to agree.

@srowen
Copy link
Member

srowen commented May 5, 2016

Jenkins test this please

@SparkQA
Copy link

SparkQA commented May 5, 2016

Test build #57884 has finished for PR 12931 at commit ae4d603.

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

@andrewor14
Copy link
Contributor

Merging into master 2.0

asfgit pushed a commit that referenced this pull request May 5, 2016
## What changes were proposed in this pull request?
set log level to debug when check shouldRollover

## How was this patch tested?
It's tested manually.

Author: depend <[email protected]>

Closes #12931 from depend/master.

(cherry picked from commit 5c47db0)
Signed-off-by: Andrew Or <[email protected]>
@asfgit asfgit closed this in 5c47db0 May 5, 2016
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.

4 participants