commit 30f45fbb0d1512a5be106e1a15744c7fa5450f1a
author Sho SHIMIZU <sshimizu@us.fujitsu.com> Wed Dec 03 18:13:26 2014 -0800
committer Gerrit Code Review <gerrit@onlab.us> Thu Dec 04 02:53:26 2014 +0000
tree 9712bb94f8c6e14a6dde1743861eba38e0a24a17
parent e9a3e2223405d2de6a00ac500af55a88fc23c163

Refactor: add static factory method in BatchWrite, instead of in IntentStore

Change-Id: Ic0cecc4b3cc4facbc469d2eb87a36c4d5166a6f2

core/api/src/main/java/org/onosproject/net/intent/BatchWrite.java

diff --git a/core/api/src/main/java/org/onosproject/net/intent/BatchWrite.java b/core/api/src/main/java/org/onosproject/net/intent/BatchWrite.java
index e030036..d528e28 100644
--- a/core/api/src/main/java/org/onosproject/net/intent/BatchWrite.java
+++ b/core/api/src/main/java/org/onosproject/net/intent/BatchWrite.java
@@ -24,7 +24,7 @@
 
 import static com.google.common.base.Preconditions.checkNotNull;
 
-public class BatchWrite {
+public final class BatchWrite {
 
     public enum OpType {
         CREATE_INTENT,
@@ -36,6 +36,17 @@
 
     List<Operation> operations = new ArrayList<>();
 
+    private BatchWrite() {}
+
+    /**
+     * Returns a new empty batch write operation builder.
+     *
+     * @return BatchWrite
+     */
+    public static BatchWrite newInstance() {
+        return new BatchWrite();
+    }
+
     public List<Operation> operations() {
         return Collections.unmodifiableList(operations);
     }

core/api/src/main/java/org/onosproject/net/intent/IntentStore.java

diff --git a/core/api/src/main/java/org/onosproject/net/intent/IntentStore.java b/core/api/src/main/java/org/onosproject/net/intent/IntentStore.java
index 129a15b..452cd97 100644
--- a/core/api/src/main/java/org/onosproject/net/intent/IntentStore.java
+++ b/core/api/src/main/java/org/onosproject/net/intent/IntentStore.java
@@ -110,16 +110,6 @@
      */
     void removeInstalledIntents(IntentId intentId);
 
-
-    /**
-     * Returns a new empty batch write operation buider.
-     *
-     * @return BatchWrite
-     */
-    default BatchWrite newBatchWrite() {
-        return new BatchWrite();
-    }
-
     /**
      * Execute writes in a batch.
      *