Allon Mureinik has uploaded a new change for review.

Change subject: core: Remove printStackTrace from VmHandler
......................................................................

core: Remove printStackTrace from VmHandler

Removed a useless call to e.printStackTrace() - the error is already
logged, and stderr isn't viewable by anyone.

Change-Id: I7ec33e5c8cb319fed39d6934fae63b413d2b378a
Signed-off-by: Allon Mureinik <[email protected]>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java
1 file changed, 0 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/23/33423/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java
index 1db25db..17be1c9 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java
@@ -828,7 +828,6 @@
             try {
                 isEnabled = (Boolean) 
field.get(objectWithEditableDeviceFields);
             } catch (IllegalAccessException | ClassCastException e) {
-                e.printStackTrace();
                 log.warn("VmHandler:: isUpdateValidForVmDevices: Reflection 
error");
             }
 


-- 
To view, visit http://gerrit.ovirt.org/33423
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I7ec33e5c8cb319fed39d6934fae63b413d2b378a
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to