Skip to content

[Trivial] Move evodb_cache Memory Usage logging#2322

Merged
random-zebra merged 1 commit intoPIVX-Project:masterfrom
PeterL73:2021-move-evodb_cache-to-updatetip-logging
Apr 20, 2021
Merged

[Trivial] Move evodb_cache Memory Usage logging#2322
random-zebra merged 1 commit intoPIVX-Project:masterfrom
PeterL73:2021-move-evodb_cache-to-updatetip-logging

Conversation

@PeterL73
Copy link

The log message for evodb_cache Memory Usage is missing a newline character, therefore the next log message is concatenated.
Combining evodb_cache Memory Usage with previous UpdateTip log message.

Before
2021-04-18 07:36:46 UpdateTip: new best=c58f3103a1614981c5ff8fa9738ca2fba90dfcabd80e054b1b5111e5bf439e62 height=2810012 version=8 log2_work=70.0675345724608150 tx=6589582 date=2021-04-18 07:36:45 progress=1.000000 cache=0.0MiB(189txo)
2021-04-18 07:36:46 UpdateTip: evodb_cache=0.0MiBProcessNewBlock : ACCEPTED Block 2810012 in 50 milliseconds with size=686

After
2021-04-18 08:01:07 UpdateTip: new best=afc6d575259aa012ce168872e788c7df8f683a475e30494bd5eccb03b7b32e1b height=2810042 version=8 log2_work=70.0675359284255421 tx=6589646 date=2021-04-18 08:00:45 progress=1.000000 cache=0.0MiB(5txo) evodb_cache=0.0MiB
2021-04-18 08:01:07 ProcessNewBlock : ACCEPTED Block 2810042 in 70 milliseconds with size=483

@random-zebra
Copy link

Thanks. This was fixed in the first commit of #2273.

@random-zebra
Copy link

We can also fix this directly here, since it's trivial, and will rebase #2273 afterwards.

Copy link

@furszy furszy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK 55daf4a

Copy link

@random-zebra random-zebra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK 55daf4a and merging...

@random-zebra random-zebra merged commit 823e728 into PIVX-Project:master Apr 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Trivial extremely simple issues Util / log / libs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants