Alon Bar-Lev has posted comments on this change.
Change subject: host-deploy: add Closeable interface to our objects
......................................................................
Patch Set 1:
(1 comment)
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsDeploy.java
Line 872: try {
Line 873: _control.close();
Line 874: }
Line 875: catch (IOException ee) {
Line 876: log.error("Error during close", e);
well, based on the spec, the close() should throw IOException... so I follow
the spec and ignore exception where close is used.
do you think we can swallow exception at close?
even if we do we still ends up with throws IOException on close and must deal.
Line 877: }
Line 878: }
Line 879: }
Line 880:
--
To view, visit http://gerrit.ovirt.org/20796
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iae58c623eb1f3acabe2404022b9bdbae32095bbb
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Sahina Bose <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
