Michael Blow has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/1611
Change subject: Fix Large Page Budget Check
......................................................................
Fix Large Page Budget Check
The logic for ensuring for large page budget is off by one; fix the
comparison.
Change-Id: I047d52f53ae26febc8e8f0a54de557409276eb91
---
M
hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/ClockPageReplacementStrategy.java
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/11/1611/1
diff --git
a/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/ClockPageReplacementStrategy.java
b/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/ClockPageReplacementStrategy.java
index c12f288..872ac35 100644
---
a/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/ClockPageReplacementStrategy.java
+++
b/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/ClockPageReplacementStrategy.java
@@ -209,7 +209,7 @@
}
private void ensureBudgetForLargePages(int delta) {
- while (numPages.get() + delta >= maxAllowedNumPages) {
+ while (numPages.get() + delta > maxAllowedNumPages) {
ICachedPageInternal victim = findVictimByEviction();
if (victim != null) {
final int victimMultiplier = victim.getFrameSizeMultiplier();
--
To view, visit https://asterix-gerrit.ics.uci.edu/1611
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I047d52f53ae26febc8e8f0a54de557409276eb91
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow <[email protected]>
