Repository: giraph
Updated Branches:
  refs/heads/trunk f31e9a328 -> 25d912cd9


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/trunk
Commit: 25d912cd910712a7c1e7da98db657f6dce877e7e
Parents: f31e9a3
Author: Pavan Kumar <[email protected]>
Authored: Mon Jul 14 10:18:37 2014 -0700
Committer: Pavan Kumar <[email protected]>
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);
+    }
+  }
 }

Reply via email to