From b2d39347a879e13d2153b07d1c869539e5144d25 Mon Sep 17 00:00:00 2001 From: Robert Varga Date: Tue, 12 Apr 2016 16:47:49 +0200 Subject: [PATCH 1/1] Enforce memory percentage Setting a negative percentage does not make sense. Make sure we validate them in setters. Change-Id: I4e0eb0a6ea47b3dda897946cc172899996e301f3 Signed-off-by: Robert Varga --- .../controller/cluster/datastore/DatastoreContext.java | 2 ++ 1 file changed, 2 insertions(+) diff --git a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/DatastoreContext.java b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/DatastoreContext.java index 6af9622199..cf14ebe28d 100644 --- a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/DatastoreContext.java +++ b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/DatastoreContext.java @@ -220,6 +220,8 @@ public class DatastoreContext { } private void setSnapshotDataThresholdPercentage(int shardSnapshotDataThresholdPercentage) { + Preconditions.checkArgument(shardSnapshotDataThresholdPercentage >= 0 + && shardSnapshotDataThresholdPercentage <= 100); raftConfig.setSnapshotDataThresholdPercentage(shardSnapshotDataThresholdPercentage); } -- 2.36.6