Hello Kudu Jenkins, Adar Dembo,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/14650
to look at the new patch set (#2).
Change subject: wip KUDU-2929: don't do nothing when under memory pressure
......................................................................
wip KUDU-2929: don't do nothing when under memory pressure
When we're under memory pressure, we may exit early from considering
ops if there are none that we know will free memory. This might be
because we don't want to spend more memory performing an op when we are
already under memory pressure.
While nice in theory, I don't think this holds up in production. Yes, we
want to prioritize ops that definitely free memory over those that might
not, but doing _something_ seems better than nothing.
So rather than returning early if there aren't ops that free memory,
let's continue walking through valid ops (in order of what frees the
most WALs, then what frees the most on-disk data, then what has the best
perf improvement).
wip: I need to test this on a real cluster.
Change-Id: I030f9ef379af501fe7bd2f42906ec2f9ea16dbde
---
M src/kudu/util/maintenance_manager-test.cc
M src/kudu/util/maintenance_manager.cc
2 files changed, 27 insertions(+), 10 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/50/14650/2
--
To view, visit http://gerrit.cloudera.org:8080/14650
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I030f9ef379af501fe7bd2f42906ec2f9ea16dbde
Gerrit-Change-Number: 14650
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
