-
Notifications
You must be signed in to change notification settings - Fork 206
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add BeanInfoCache support #115
Conversation
This reverts commit 886ef36.
Added BeanInfo Cache support for BeanUtil.java https://github.com/apache/axis2-java/commit/d97f24e0dd1d369e7753c5e7b213f4485c5d1382
* @return the cache instance; must not be <code>null</code> | ||
*/ | ||
BeanInfoCache getBeanInfoCache(); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing eof line
} | ||
return cache.getBeanInfo(beanClass, stopClass); | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing eof new line
File needs to end with a new line
@@ -0,0 +1,122 @@ | |||
/* | |||
* Licensed to the Apache Software Foundation (ASF) under one |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
New classes need to be released under the WSO2 license.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kasunbg : The reason for having this license is because these classes are from the fix from the Axis2 repo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alright. Yes, in that case, we need to license it to ASF.
@@ -0,0 +1,33 @@ | |||
/* | |||
* Licensed to the Apache Software Foundation (ASF) under one |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
New classes need to be released under the WSO2 license.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kasunbg : The reason for having this license is because these classes are from the fix from the Axis2 repo.
This contains the following changes:
Resolves #113