SpyImpl print error log

core/src/main/java/com/taobao/arthas/core/advisor/SpyImpl.java

@@ -41,9 +41,7 @@ public class SpyImpl extends AbstractSpy {
                     }
                     adviceListener.before(clazz, methodName, methodDesc, target, args);
                 } catch (Throwable e) {
-                    if (logger.isDebugEnabled()) {
-                        logger.error("class: {}, methodInfo: {}", clazz.getName(), methodInfo, e);
-                    }
+                    logger.error("class: {}, methodInfo: {}", clazz.getName(), methodInfo, e);
                 }
             }
         }
@@ -68,9 +66,7 @@ public class SpyImpl extends AbstractSpy {
                     }
                     adviceListener.afterReturning(clazz, methodName, methodDesc, target, args, returnObject);
                 } catch (Throwable e) {
-                    if (logger.isDebugEnabled()) {
-                        logger.error("class: {}, methodInfo: {}", clazz.getName(), methodInfo, e);
-                    }
+                    logger.error("class: {}, methodInfo: {}", clazz.getName(), methodInfo, e);
                 }
             }
         }
@@ -94,9 +90,7 @@ public class SpyImpl extends AbstractSpy {
                     }
                     adviceListener.afterThrowing(clazz, methodName, methodDesc, target, args, throwable);
                 } catch (Throwable e) {
-                    if (logger.isDebugEnabled()) {
-                        logger.error("class: {}, methodInfo: {}", clazz.getName(), methodInfo, e);
-                    }
+                    logger.error("class: {}, methodInfo: {}", clazz.getName(), methodInfo, e);
                 }
             }
         }
@@ -122,9 +116,7 @@ public class SpyImpl extends AbstractSpy {
                     final InvokeTraceable listener = (InvokeTraceable) adviceListener;
                     listener.invokeBeforeTracing(owner, methodName, methodDesc, Integer.parseInt(info[3]));
                 } catch (Throwable e) {
-                    if (logger.isDebugEnabled()) {
-                        logger.error("class: {}, invokeInfo: {}", clazz.getName(), invokeInfo, e);
-                    }
+                    logger.error("class: {}, invokeInfo: {}", clazz.getName(), invokeInfo, e);
                 }
             }
         }
@@ -149,9 +141,7 @@ public class SpyImpl extends AbstractSpy {
                     final InvokeTraceable listener = (InvokeTraceable) adviceListener;
                     listener.invokeAfterTracing(owner, methodName, methodDesc, Integer.parseInt(info[3]));
                 } catch (Throwable e) {
-                    if (logger.isDebugEnabled()) {
-                        logger.error("class: {}, invokeInfo: {}", clazz.getName(), invokeInfo, e);
-                    }
+                    logger.error("class: {}, invokeInfo: {}", clazz.getName(), invokeInfo, e);
                 }
             }
         }
@@ -178,9 +168,7 @@ public class SpyImpl extends AbstractSpy {
                     final InvokeTraceable listener = (InvokeTraceable) adviceListener;
                     listener.invokeThrowTracing(owner, methodName, methodDesc, Integer.parseInt(info[3]));
                 } catch (Throwable e) {
-                    if (logger.isDebugEnabled()) {
-                        logger.error("class: {}, invokeInfo: {}", clazz.getName(), invokeInfo, e);
-                    }
+                    logger.error("class: {}, invokeInfo: {}", clazz.getName(), invokeInfo, e);
                 }
             }
         }