-
Notifications
You must be signed in to change notification settings - Fork 29k
[SPARK-24873][YARN] Turn off spark-shell noisy log output #21784
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Test build #93111 has finished for PR 21784 at commit
|
|
cc @vanzin, @jerryshao |
| try { | ||
| val YarnAppReport(_, state, diags) = | ||
| client.monitorApplication(appId.get, logApplicationReport = true) | ||
| client.monitorApplication(appId.get, logApplicationReport = false) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, it's too verbose currently in the client mode. I remembered we only have such output in cluster mode YARN client. My only concern is that turning to false will also lose the detailed reports. I think it would be better if we still have the detailed report when state is changed.
|
Hm, yea. I don't think find this super noisy though to be honest. |
|
OK, but you can set Considering it's info level, I was thinking it's not super noisy. I am okay with disabling though. I got your point. Let me defer this to @vanzin and @jerryshao. |
|
@vanzin WDYT about this? |
HyukjinKwon
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
Also, mind adding |
|
Merged to master. |
|
what? I think we need to add a switch. |
|
? I think we don't need a switch. |
|
Maybe we could consider avoiding this logs in shell specifically. Adding a switch for disable/enable logs sounds an overkill. |
|
But for some spark-submit applications, I want these Application report for information. |
|
It has never been printed before, right? I think we can consider to turn it on specifically for spark-submit applications although I am not fully sure if it's something worth doing so yet. |
|
We need to listen to @vanzin opinion. |
|
Adding a configuration to control some logs sounds an overkill. I wouldn't go for this way. |
|
Logging is already configurable; a switch is overkill. This seems fine. |
|
Belated +1; I didn't mean to make the output noisier, probably just flipped the value for debugging and forgot about it. |

What changes were proposed in this pull request?
SPARK-24182 changed the
logApplicationReportfromfalsetotrue. This pr revert it tofalse. otherwisespark-shellwill show noisy log output:Closes #21827
How was this patch tested?
manual tests