Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Enhance metrics #2209

Closed
hokiegeek2 opened this issue Mar 11, 2023 · 1 comment · Fixed by #2332
Closed

Enhance metrics #2209

hokiegeek2 opened this issue Mar 11, 2023 · 1 comment · Fixed by #2332
Assignees
@hokiegeek2
Labels
enhancement New feature or request

Comments

@hokiegeek2
Copy link
Contributor

Feature Description
Enhance the Arkouda metrics by adding the following:

  1. Total memory used per command
  2. Total execution time in seconds per command

Additional context
The arkouda_contrib arkouda_metrics_exporter) will be updated accordingly.
@hokiegeek2 hokiegeek2 added the enhancement New feature or request label Mar 11, 2023
@hokiegeek2 hokiegeek2 self-assigned this Mar 11, 2023
hokiegeek2 added a commit to hokiegeek2/arkouda that referenced this issue Apr 1, 2023
@hokiegeek2
enabled support for total memory used and total response times Bears-…
56e84e7 
…R-Us#2209
@hokiegeek2
Copy link
Contributor Author

implemented both requirements, example logging is shown below:

**
image
**

hokiegeek2 added a commit to hokiegeek2/arkouda that referenced this issue Apr 2, 2023
@hokiegeek2
updated memory metrics to capture GB of RAM per command Bears-R-Us#2209
0b701cb
@hokiegeek2 hokiegeek2 mentioned this issue Apr 2, 2023
Merged
hokiegeek2 added a commit to hokiegeek2/arkouda that referenced this issue Apr 4, 2023
@hokiegeek2
updated per PR feedback, added chapeldoc comments, standarized logic B…
9f1a15b 
…ears-R-Us#2209
hokiegeek2 added a commit to hokiegeek2/arkouda that referenced this issue Apr 4, 2023
@hokiegeek2
reverse code change made per 1.30.0 deprecation warning Bears-R-Us#2209
cf71e71
hokiegeek2 added a commit to hokiegeek2/arkouda that referenced this issue Apr 6, 2023
@hokiegeek2
fixed merge conflict, tweaked log messages Bears-R-Us#2209
891bfb7
hokiegeek2 added a commit to hokiegeek2/arkouda that referenced this issue Apr 7, 2023
@hokiegeek2
reverted refactoring, added pull logic for new metrics Bears-R-Us#2209
9b913a5
Ethan-DeBandi99 pushed a commit that referenced this issue Apr 7, 2023
@hokiegeek2
2209 enhance Arkouda metrics (#2332)
80c4fe6 
* enabled support for total memory used and total response times #2209

* updated memory metrics to capture GB of RAM per command #2209

* updated per PR feedback, added chapeldoc comments, standarized logic  #2209

* reverse code change made per 1.30.0 deprecation warning #2209

* reverted refactoring, added pull logic for new metrics #2209
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hokiegeek2 hokiegeek2

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

2209 enhance Arkouda metrics hokiegeek2/arkouda
1 participant
@hokiegeek2