Michael Blow has submitted this change and it was merged. Change subject: Decrement Large Pages When Releasing From VBC ......................................................................
Decrement Large Pages When Releasing From VBC Change-Id: Ic61951fdc48f501d8b006f89f5d77791cd072ae3 Reviewed-on: https://asterix-gerrit.ics.uci.edu/1610 Reviewed-by: Ian Maxon <[email protected]> Tested-by: Jenkins <[email protected]> --- M hyracks-fullstack/hyracks/hyracks-storage-am-lsm-common/src/main/java/org/apache/hyracks/storage/am/lsm/common/impls/VirtualBufferCache.java 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Ian Maxon: Looks good to me, approved Jenkins: Verified Objections: Jenkins: Violations found diff --git a/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-common/src/main/java/org/apache/hyracks/storage/am/lsm/common/impls/VirtualBufferCache.java b/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-common/src/main/java/org/apache/hyracks/storage/am/lsm/common/impls/VirtualBufferCache.java index fce0b74..27d879c 100644 --- a/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-common/src/main/java/org/apache/hyracks/storage/am/lsm/common/impls/VirtualBufferCache.java +++ b/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-common/src/main/java/org/apache/hyracks/storage/am/lsm/common/impls/VirtualBufferCache.java @@ -105,6 +105,9 @@ VirtualPage curr = bucket.cachedPage; while (curr != null) { if (BufferedFileHandle.getFileId(curr.dpid()) == fileId) { + if (curr.getFrameSizeMultiplier() > 1) { + largePages.getAndAdd(-curr.getFrameSizeMultiplier()); + } if (prev == null) { bucket.cachedPage = curr.next(); curr.reset(); -- To view, visit https://asterix-gerrit.ics.uci.edu/1610 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic61951fdc48f501d8b006f89f5d77791cd072ae3 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow <[email protected]> Gerrit-Reviewer: Ian Maxon <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Michael Blow <[email protected]>
