Eliminate ShardedDOMDataTreeWriteTransaction.doSubmit()'s return 86/45886/4
authorRobert Varga <rovarga@cisco.com>
Tue, 20 Sep 2016 12:59:19 +0000 (14:59 +0200)
committerRobert Varga <rovarga@cisco.com>
Wed, 21 Sep 2016 11:50:41 +0000 (13:50 +0200)
Returned future is not used anywhere, hence we can easily remove it.

Change-Id: I4ead4da32d1c2658d5c88f01534b5452230068e7
Signed-off-by: Robert Varga <rovarga@cisco.com>
dom/mdsal-dom-broker/src/main/java/org/opendaylight/mdsal/dom/broker/ShardedDOMDataTreeWriteTransaction.java

index 0e1180ab12eb8a0c4bd6456499077308800a8db6..08f359d2161e9dcb64f2c9042b45a6f62ab93710 100644 (file)
@@ -122,8 +122,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(
@@ -133,22 +132,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) {