-
Notifications
You must be signed in to change notification settings - Fork 18
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
DM-39516: Extend Qserv Web Dashboard to allow monitoring active queries at Czar and worker database servers #803
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Also added the utility class to handle simple query cases.
jgates108
approved these changes
Aug 16, 2023
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.
Looks good, just very minor comments.
Simplified error reporting. Reduced code duplication.
Added a function for getting info on the ongoing database activities as reported by the query SHOW [FULL] PROCESSLIST.
Added the REST services to report database activities at Czar and Qserv workers. Incremented the version number of the REST API.
The identifiers are captured for the worker task monitoring purposes. They are used by the Web Dashboard for associating task activities with the corresponding MySQL queries issued by the tasks. The task monitoring page of the Dashboard has been extended to display a value of the identifier (if it was set for the task).
Introduced an additional state STARTED to differentiate between two events - when a task was started by a scheduler from the actual start time of the corresponding MySQL query. The change is meant to improve the monitoring of the worker tasks. The corresponding monitoring page of the Web Dashboard was modified as well.
iagaponenko
force-pushed
the
tickets/DM-39516
branch
from
August 17, 2023 00:58
2a9d617
to
cefbb18
Compare
iagaponenko
changed the title
Tickets/dm 39516
DM-39516: Extend Qserv Web Dashboard to allow monitoring active queries at Czar and worker database servers
Aug 18, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.