diff options
author | Maros Marsalek <mmarsale@cisco.com> | 2016-08-31 15:16:38 +0200 |
---|---|---|
committer | Maros Marsalek <mmarsale@cisco.com> | 2016-08-31 14:27:59 +0000 |
commit | 2cdd56d31c4514d7d7000ce7c7b6417ce53e2be2 (patch) | |
tree | b474759ff9bdd3b0d310377e8435f0182454b779 /infra/data-impl/src/main/java/io/fd/honeycomb/data/impl/WriteTransaction.java | |
parent | dace04173cd4846c3cfbcf44ede9cc7081909588 (diff) |
Fixing sonar issues #1
Change-Id: I4c27d0ad2fd1655db2237ae914361f029093084c
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>
Diffstat (limited to 'infra/data-impl/src/main/java/io/fd/honeycomb/data/impl/WriteTransaction.java')
-rw-r--r-- | infra/data-impl/src/main/java/io/fd/honeycomb/data/impl/WriteTransaction.java | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/infra/data-impl/src/main/java/io/fd/honeycomb/data/impl/WriteTransaction.java b/infra/data-impl/src/main/java/io/fd/honeycomb/data/impl/WriteTransaction.java index 6bc6b1b4b..2cbe1ec8b 100644 --- a/infra/data-impl/src/main/java/io/fd/honeycomb/data/impl/WriteTransaction.java +++ b/infra/data-impl/src/main/java/io/fd/honeycomb/data/impl/WriteTransaction.java @@ -75,16 +75,18 @@ final class WriteTransaction implements DOMDataWriteTransaction { } private void handleOperation(final LogicalDatastoreType store, - final Consumer<DataModification> r) { + final Consumer<DataModification> modificationHandler) { switch (store) { case CONFIGURATION: checkArgument(configModification != null, "Modification of %s is not supported", store); - r.accept(configModification); + modificationHandler.accept(configModification); break; case OPERATIONAL: checkArgument(operationalModification != null, "Modification of %s is not supported", store); - r.accept(operationalModification); + modificationHandler.accept(operationalModification); break; + default: + throw new IllegalArgumentException("Unable to handle operation for type " + store); } } |