Revert "Merge #10126: Compensate for memory peak at flush time"
ClosedPublic
Actions

Authored by jasonbcox on May 9 2019, 19:41.

Details

Reviewers

deadalnix
Fabien
markblundeberg

Group Reviewers

Restricted Project

Commits

rSTAGINGb809b0250876: Revert "Merge #10126: Compensate for memory peak at flush time"
rABCb809b0250876: Revert "Merge #10126: Compensate for memory peak at flush time"

Summary

This reverts commit 7bd410c3c23067fcef2c6671676c104fbb8e974a.
Reverts D2999

After backporting PR10126 and then looking into PR10133 (https://github.com/bitcoin/bitcoin/pull/10133/files) it's clear that we've already backported up to this point and that re-implementing PR10126 is a step backwards.

Test Plan

make

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jasonbcox created this revision.May 9 2019, 19:41

Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 9 2019, 19:41

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

jasonbcox edited the summary of this revision. (Show Details)May 9 2019, 19:43

jasonbcox added a reverted change: D2999: Merge #10126: Compensate for memory peak at flush time.

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing passed.

Harbormaster completed remote builds in B5784: Diff 8569.May 9 2019, 19:56

I see, you mean that D1049 backported PR11022 already?

This revision is now accepted and ready to land.May 9 2019, 23:52

Closed by commit rABCb809b0250876: Revert "Merge #10126: Compensate for memory peak at flush time" (authored by jasonbcox). · Explain WhyMay 10 2019, 01:09

This revision was automatically updated to reflect the committed changes.

jasonbcox added a commit: rSTAGINGb809b0250876: Revert "Merge #10126: Compensate for memory peak at flush time".Sep 18 2019, 23:27

Revision Contents
Changeset List

Path

Size

src/

validation.cpp

3 lines

Diff 8574

View Options

Revert "Merge #10126: Compensate for memory peak at flush time"ClosedPublicActions