Skip to content
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

Include DescriptorFields (method parameter metadata), indirection to support dynamic resolution (EJBs) #1

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

bvarner
Copy link

@bvarner bvarner commented Dec 11, 2014

Greetings!

I'd like to submit my changes upstream. I'm about to publish a cdi extension which can publish cdi and jee beans as MBeans in another repo and figured it would be worth pushing this upstream.

The layer of indirection was added so that alternative implementations can resolve delegates from proxy info (say, JNDI lookups in the case of EJBs) and done in a manner that shouldn't harm the operation of the core library.

Regards,
-Bryan

Bryan Varner and others added 6 commits November 8, 2013 17:19
…should be able to name parameters with this.
…oke JMX operations upon.

Long term, it would make sense to use the BeanUtils bits to find the getter / setters, and would make sense to add extension points to inspect other class-level annotations while making decisions about Attribute / Method accessiblility.
@buildhive
Copy link

German Escobar » jmx-annotations #10 SUCCESS
This pull request looks good
(what's this?)

@bvarner
Copy link
Author

bvarner commented Dec 15, 2014

Any change this will get accepted?

@bvarner
Copy link
Author

bvarner commented Dec 15, 2015

One last try before I give up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants