-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Improve AcAC to ActorMonitoring #11584
base: main
Are you sure you want to change the base?
Conversation
@spraza -- I may not have enough time to finish this within my time frame, it would be more than welcomed if you are interested to spend some time on it. |
Result of foundationdb-pr-clang-arm on Linux CentOS 7
|
Result of foundationdb-pr-clang on Linux CentOS 7
|
Result of foundationdb-pr-cluster-tests on Linux CentOS 7
|
Result of foundationdb-pr on Linux CentOS 7
|
Result of foundationdb-pr-clang-ide on Linux CentOS 7
|
Result of foundationdb-pr-macos-m1 on macOS Ventura 13.x
|
Result of foundationdb-pr-macos on macOS Ventura 13.x
|
@@ -0,0 +1,20 @@ | |||
# Actor Monitor | |||
|
|||
# Advanced Materials is a peer-reviewed journal covering material topics. Its impact factor is 29.4(2022) |
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.
What's this?
*/ | ||
|
||
#include "flow/ActorContext.h" | ||
|
||
#ifdef WITH_ACAC |
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.
Should this be the new macro (ACTOR_MONITORING
)? Currently majority of the code is pre-processed out. If I change this code to use the new macro, then the code is not pre-processed out but then it also does not compile.
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.
They have to be rewritten to reflect the changes. I have only implemented dumpActorCallBacktrace
but not the others. But it would be great if you can try dumpActorCallBacktrace
in some ACTORs during the simulation tests.
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.
So... again, if you do not have time, I can make it later next month.
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.
So... again, if you do not have time, I can make it later next month.
Ok, I can review this next month once you update it
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.
So... again, if you do not have time, I can make it later next month.
Ok, I can review this next month once you update it
This code is runnable though, if you got time can you put a dumpActorCallBacktrace
to your code and see if it generates backtrace? At least it worked for me (I tested it on storage server/TLog)
This is a draft of ActorMonitoring code.
The improvements:
The missing parts:
Code-Reviewer Section
The general pull request guidelines can be found here.
Please check each of the following things and check all boxes before accepting a PR.
For Release-Branches
If this PR is made against a release-branch, please also check the following:
release-branch
ormain
if this is the youngest branch)