You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@giraph.apache.org by rv...@apache.org on 2014/10/26 02:21:57 UTC

[14/47] git commit: updated refs/heads/release-1.1 to 4c139ee

GIRAPH-929: setIfUnset for EnumConfOption (pavanka)


Project: http://git-wip-us.apache.org/repos/asf/giraph/repo
Commit: http://git-wip-us.apache.org/repos/asf/giraph/commit/25d912cd
Tree: http://git-wip-us.apache.org/repos/asf/giraph/tree/25d912cd
Diff: http://git-wip-us.apache.org/repos/asf/giraph/diff/25d912cd

Branch: refs/heads/release-1.1
Commit: 25d912cd910712a7c1e7da98db657f6dce877e7e
Parents: f31e9a3
Author: Pavan Kumar <pa...@fb.com>
Authored: Mon Jul 14 10:18:37 2014 -0700
Committer: Pavan Kumar <pa...@fb.com>
Committed: Mon Jul 14 10:18:37 2014 -0700

----------------------------------------------------------------------
 CHANGELOG                                               |  2 ++
 .../java/org/apache/giraph/conf/EnumConfOption.java     | 12 ++++++++++++
 2 files changed, 14 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/giraph/blob/25d912cd/CHANGELOG
----------------------------------------------------------------------
diff --git a/CHANGELOG b/CHANGELOG
index 0263749..e1c1227 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -1,6 +1,8 @@
 Giraph Change Log
 
 Release 1.1.0 - unreleased
+  GIRAPH-929: setIfUnset for EnumConfOption (pavanka)  
+
   GIRAPH-912: Support succinct representation of messages in messagestores (pavanka)
 
   GIRAPH-903: Detect crashes of Netty threads (edunov via pavanka)

http://git-wip-us.apache.org/repos/asf/giraph/blob/25d912cd/giraph-core/src/main/java/org/apache/giraph/conf/EnumConfOption.java
----------------------------------------------------------------------
diff --git a/giraph-core/src/main/java/org/apache/giraph/conf/EnumConfOption.java b/giraph-core/src/main/java/org/apache/giraph/conf/EnumConfOption.java
index 2a65ec0..a0effe1 100644
--- a/giraph-core/src/main/java/org/apache/giraph/conf/EnumConfOption.java
+++ b/giraph-core/src/main/java/org/apache/giraph/conf/EnumConfOption.java
@@ -95,4 +95,16 @@ public class EnumConfOption<T extends Enum<T>> extends AbstractConfOption {
   public void set(Configuration conf, Enum<T> value) {
     conf.set(getKey(), value.name());
   }
+
+  /**
+   * Set value if it's not already present
+   *
+   * @param conf Configuration
+   * @param value to set
+   */
+  public void setIfUnset(Configuration conf, Enum<T> value) {
+    if (!contains(conf)) {
+      set(conf, value);
+    }
+  }
 }