ert icon indicating copy to clipboard operation
ert copied to clipboard

Increase qstat performance by adding -E option

Open JHolba opened this issue 1 year ago • 1 comments

Using the -E option causes qstat to group job status calls going to the same server together. This makes qstat orders of magnitude faster for calls with many job ids.

  • [x] PR title captures the intent of the changes, and is fitting for release notes.
  • [x] Added appropriate release note label
  • [x] Commit history is consistent and clean, in line with the contribution guidelines.
  • [x] Make sure tests pass locally (after every commit!)

When applicable

  • [ ] When there are user facing changes: Updated documentation
  • [ ] New behavior or changes to existing untested code: Ensured that unit tests are added (See Ground Rules).
  • [ ] Large PR: Prepare changes in small commits for more convenient review
  • [ ] Bug fix: Add regression test for the bug
  • [ ] Bug fix: Create Backport PR to latest release

JHolba avatar Apr 05 '24 08:04 JHolba

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.50%. Comparing base (a1f9feb) to head (1a79da0). Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7573      +/-   ##
==========================================
- Coverage   85.13%   84.50%   -0.63%     
==========================================
  Files         387      388       +1     
  Lines       23232    23323      +91     
  Branches      877      878       +1     
==========================================
- Hits        19778    19709      -69     
- Misses       3340     3507     +167     
+ Partials      114      107       -7     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov-commenter avatar Apr 05 '24 11:04 codecov-commenter

Will this solve #5453?

jonathan-eq avatar Apr 10 '24 07:04 jonathan-eq