Eliminate ShardedDOMDataTreeWriteTransaction.doSubmit()'s return 60/46360/1
authorRobert Varga <rovarga@cisco.com>
Tue, 20 Sep 2016 12:59:19 +0000 (14:59 +0200)
committerRobert Varga <rovarga@cisco.com>
Mon, 3 Oct 2016 11:36:17 +0000 (13:36 +0200)
Returned future is not used anywhere, hence we can easily remove it.

Change-Id: I4ead4da32d1c2658d5c88f01534b5452230068e7
Signed-off-by: Robert Varga <rovarga@cisco.com>
(cherry picked from commit ada2c265698298bdab0201bb9a2c30af64376a9d)

dom/mdsal-dom-broker/src/main/java/org/opendaylight/mdsal/dom/broker/ShardedDOMDataTreeWriteTransaction.java

index fcbc9169fca9759f7ba8f5c910f2525a850ec047..272d806e884719b6be79b0d52731508614a03143 100644 (file)
@@ -129,8 +129,7 @@ final class ShardedDOMDataTreeWriteTransaction implements DOMDataTreeCursorAware
         return submitFuture;
     }
 
-    CheckedFuture<Void, TransactionCommitFailedException> doSubmit(
-            final Consumer<ShardedDOMDataTreeWriteTransaction> success,
+    void doSubmit(final Consumer<ShardedDOMDataTreeWriteTransaction> success,
             final BiConsumer<ShardedDOMDataTreeWriteTransaction, Throwable> failure) {
 
         final ListenableFuture<List<Void>> listListenableFuture = Futures.allAsList(
@@ -140,22 +139,17 @@ final class ShardedDOMDataTreeWriteTransaction implements DOMDataTreeCursorAware
                 return tx.submit();
             }).collect(Collectors.toList()));
 
-        final SettableFuture<Void> ret = SettableFuture.create();
         Futures.addCallback(listListenableFuture, new FutureCallback<List<Void>>() {
             @Override
             public void onSuccess(final List<Void> result) {
                 success.accept(ShardedDOMDataTreeWriteTransaction.this);
-                ret.set(null);
             }
 
             @Override
             public void onFailure(final Throwable exp) {
                 failure.accept(ShardedDOMDataTreeWriteTransaction.this, exp);
-                ret.setException(exp);
             }
         });
-
-        return Futures.makeChecked(ret, SUBMIT_FAILED_MAPPER);
     }
 
     void onTransactionSuccess(final Void result) {