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

Minor log fix #1031

Merged

bcipriano merged 1 commit into AcademySoftwareFoundation:master from splhack:rqd-log

Sep 12, 2021

Contributor

splhack commented Sep 10, 2021

The RQD log footer always has an extra space on the separator
Out of memory error message has an extra "MB", like 20480MBMB but the machine only has
- OpenCue/cuebot/src/main/java/com/imageworks/spcue/dispatcher/HostReportHandler.java
  
  Line 433 in 21bfbfe
  
  CueUtil.KbToMb(f.getRss()) + "MB but the machine only has " +
- KbToMb already prints MB
  
  OpenCue/cuebot/src/main/java/com/imageworks/spcue/util/CueUtil.java
  
  Line 207 in 21bfbfe
  
  return String.format("%dMB", kb / 1024);

splhack requested review from bcipriano, DiegoTavares, gregdenton, IdrisMiles, jrray, larsbijl and smith1511 as code owners

September 10, 2021 05:39


Minor log fix

34b42e3

splhack force-pushed the rqd-log branch from 216be1c to 34b42e3 Compare

September 10, 2021 16:11

bcipriano approved these changes

View reviewed changes

Collaborator

bcipriano left a comment

Thank you! That extra space always did bug me 😄

bcipriano merged commit 1cedc89 into AcademySoftwareFoundation:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment