Repository: lens
Updated Branches:
  refs/heads/master ee2bda93f -> 962c8880d


LENS-976: Change log level to debug to avoid too many warning logs in 
YAMLToStringStrategy in test cases


Project: http://git-wip-us.apache.org/repos/asf/lens/repo
Commit: http://git-wip-us.apache.org/repos/asf/lens/commit/962c8880
Tree: http://git-wip-us.apache.org/repos/asf/lens/tree/962c8880
Diff: http://git-wip-us.apache.org/repos/asf/lens/diff/962c8880

Branch: refs/heads/master
Commit: 962c8880d49caa5981b9342c48eed23f74e9d09a
Parents: ee2bda9
Author: Rajat Khandelwal <pro...@apache.org>
Authored: Tue Mar 1 07:50:18 2016 +0800
Committer: raju <r...@apache.org>
Committed: Tue Mar 1 07:50:18 2016 +0800

----------------------------------------------------------------------
 .../java/org/apache/lens/api/jaxb/YAMLToStringStrategy.java   | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/lens/blob/962c8880/lens-api/src/main/java/org/apache/lens/api/jaxb/YAMLToStringStrategy.java
----------------------------------------------------------------------
diff --git 
a/lens-api/src/main/java/org/apache/lens/api/jaxb/YAMLToStringStrategy.java 
b/lens-api/src/main/java/org/apache/lens/api/jaxb/YAMLToStringStrategy.java
index f5af1c6..3b55dfe 100644
--- a/lens-api/src/main/java/org/apache/lens/api/jaxb/YAMLToStringStrategy.java
+++ b/lens-api/src/main/java/org/apache/lens/api/jaxb/YAMLToStringStrategy.java
@@ -318,12 +318,13 @@ public class YAMLToStringStrategy extends 
JAXBToStringStrategy {
         }
       }
       if (nonStaticFields == 1) {
+        Class<?> claz = value.getClass();
+        String getterName = "get" + fieldName.substring(0, 1).toUpperCase() + 
fieldName.substring(1);
         try {
-          Object wrappedValue = value.getClass().getDeclaredMethod("get" + 
fieldName.substring(0, 1).toUpperCase()
-            + fieldName.substring(1)).invoke(value);
+          Object wrappedValue = 
claz.getDeclaredMethod(getterName).invoke(value);
           return appendNewLine(appendInternal(locator, stringBuilder, 
wrappedValue));
         } catch (NoSuchMethodException | IllegalAccessException | 
InvocationTargetException e) {
-          log.error("getter access failed. Going the usual way");
+          log.debug("getter access failed for {}#{}. Going the usual way", 
claz.getName(), getterName, e);
         }
       }
       return super.appendInternal(locator, stringBuilder, value);

Reply via email to