Code Review
/
netconf.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
Fix logging arguments
[netconf.git]
/
netconf
/
callhome-provider
/
src
/
main
/
java
/
org
/
opendaylight
/
netconf
/
callhome
/
mount
/
CallhomeStatusReporter.java
diff --git
a/netconf/callhome-provider/src/main/java/org/opendaylight/netconf/callhome/mount/CallhomeStatusReporter.java
b/netconf/callhome-provider/src/main/java/org/opendaylight/netconf/callhome/mount/CallhomeStatusReporter.java
index 728c291b786de1932c83b7bb26fc042f90bcb6ea..1f819ae0a34ffa3f0cf5a32597ed9c72d1be6c86 100644
(file)
--- a/
netconf/callhome-provider/src/main/java/org/opendaylight/netconf/callhome/mount/CallhomeStatusReporter.java
+++ b/
netconf/callhome-provider/src/main/java/org/opendaylight/netconf/callhome/mount/CallhomeStatusReporter.java
@@
-266,15
+266,15
@@
class CallhomeStatusReporter implements DataTreeChangeListener<Node>, StatusReco
commit(tx, device.key());
}
commit(tx, device.key());
}
- private static void commit(
WriteTransaction tx,
DeviceKey device) {
+ private static void commit(
final WriteTransaction tx, final
DeviceKey device) {
tx.commit().addCallback(new FutureCallback<CommitInfo>() {
@Override
tx.commit().addCallback(new FutureCallback<CommitInfo>() {
@Override
- public void onSuccess(CommitInfo result) {
+ public void onSuccess(
final
CommitInfo result) {
LOG.debug("Device {} committed", device);
}
@Override
LOG.debug("Device {} committed", device);
}
@Override
- public void onFailure(Throwable cause) {
+ public void onFailure(
final
Throwable cause) {
LOG.warn("Failed to commit device {}", device, cause);
}
}, MoreExecutors.directExecutor());
LOG.warn("Failed to commit device {}", device, cause);
}
}, MoreExecutors.directExecutor());
@@
-287,7
+287,7
@@
class CallhomeStatusReporter implements DataTreeChangeListener<Node>, StatusReco
try {
return devicesFuture.get().orElse(null);
} catch (ExecutionException | InterruptedException e) {
try {
return devicesFuture.get().orElse(null);
} catch (ExecutionException | InterruptedException e) {
- LOG.error("Error trying to read the whitelist devices
: {}
", e);
+ LOG.error("Error trying to read the whitelist devices", e);
return null;
}
}
return null;
}
}