Uh oh!
There was an error while loading. Please reload this page.
[SPARK-2952] Enable logging actor messages at DEBUG level - #1870
Conversation
SparkQA
commented
Aug 10, 2014
QA tests have started for PR 1870. This patch merges cleanly. |
SparkQA
commented
Aug 10, 2014
QA results for PR 1870: |
SparkQA
commented
Aug 10, 2014
QA tests have started for PR 1870. This patch merges cleanly. |
SparkQA
commented
Aug 10, 2014
QA results for PR 1870: |
rxin
commented
Aug 10, 2014
@andrewor14 is the SparkSubmit test flaky? I've seen it fail a few times already. |
rxin
commented
Aug 10, 2014
Jenkins, retest this please. |
This helps us log UpdateBlockInfo properly once #1870 is merged. Author: Reynold Xin <rxin@apache.org> Closes#1872 from rxin/UpdateBlockInfo and squashes the following commits: 0cee1c2 [Reynold Xin] Turn UpdateBlockInfo into case class. (cherry picked from commit 482c5af) Signed-off-by: Reynold Xin <rxin@apache.org>
andrewor14
commented
Aug 10, 2014
Yeah that and the |
pwendell
commented
Aug 11, 2014
LGTM - thanks Reynold! |
rxin
commented
Aug 11, 2014
Thanks. Merging in master & branch-1.1 |
Example messages: ``` 14/08/09 21:37:01 DEBUG BlockManagerMasterActor: [actor] received message RegisterBlockManager(BlockManagerId(0, rxin-mbp, 58092, 0),278302556,Actor[akka.tcp://spark@rxin-mbp:58088/user/BlockManagerActor1#-63596539]) from Actor[akka.tcp://spark@rxin-mbp:58088/temp/$c] 14/08/09 21:37:01 DEBUG BlockManagerMasterActor: [actor] handled message (0.279 ms) RegisterBlockManager(BlockManagerId(0, rxin-mbp, 58092, 0),278302556,Actor[akka.tcp://spark@rxin-mbp:58088/user/BlockManagerActor1#-63596539]) from Actor[akka.tcp://spark@rxin-mbp:58088/temp/$c] ``` cc @mengxr@tdas@pwendell Author: Reynold Xin <rxin@apache.org> Closes#1870 from rxin/actorLogging and squashes the following commits: c531ee5 [Reynold Xin] Added license header for ActorLogReceive. f6b1ebe [Reynold Xin] [SPARK-2952] Enable logging actor messages at DEBUG level (cherry picked from commit 3733866) Signed-off-by: Reynold Xin <rxin@apache.org>
mridulm
commented
Aug 11, 2014
Just saw this as part of the close, sorry for the late comment. Also, some of the INFO messages which are useful have now become DEBUG ? Makes it slightly harder to pinpoint an issue (ex: "Stopping BlockManagerMaster"). Btw, if we are logging all messages, would it not be better to use TRACE instead of DEBUG ? |
rxin
commented
Aug 11, 2014
I thought about that in this PR actually. Since in log4j you can configure what you want, it seems pretty easy to just enable debug logging for a specific package. |
mridulm
commented
Aug 11, 2014
Unfortunately, in most cases, we wont know what the issue is other than bug hunting in the logs. |
rxin
commented
Aug 12, 2014
Yea I'm fine turning that into trace. Even better, if we can configure what level to log those on, that'd be great. Do you want to submit a pull request? |
This helps us log UpdateBlockInfo properly once apache#1870 is merged. Author: Reynold Xin <rxin@apache.org> Closesapache#1872 from rxin/UpdateBlockInfo and squashes the following commits: 0cee1c2 [Reynold Xin] Turn UpdateBlockInfo into case class.
Example messages: ``` 14/08/09 21:37:01 DEBUG BlockManagerMasterActor: [actor] received message RegisterBlockManager(BlockManagerId(0, rxin-mbp, 58092, 0),278302556,Actor[akka.tcp://spark@rxin-mbp:58088/user/BlockManagerActor1#-63596539]) from Actor[akka.tcp://spark@rxin-mbp:58088/temp/$c] 14/08/09 21:37:01 DEBUG BlockManagerMasterActor: [actor] handled message (0.279 ms) RegisterBlockManager(BlockManagerId(0, rxin-mbp, 58092, 0),278302556,Actor[akka.tcp://spark@rxin-mbp:58088/user/BlockManagerActor1#-63596539]) from Actor[akka.tcp://spark@rxin-mbp:58088/temp/$c] ``` cc @mengxr@tdas@pwendell Author: Reynold Xin <rxin@apache.org> Closesapache#1870 from rxin/actorLogging and squashes the following commits: c531ee5 [Reynold Xin] Added license header for ActorLogReceive. f6b1ebe [Reynold Xin] [SPARK-2952] Enable logging actor messages at DEBUG level
Example messages:
cc @mengxr@tdas@pwendell