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.

Sign up for GitHub

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

Jump to bottom

Implementing second level classLoading and JSonProvider #4

Open
wants to merge 1 commit into
base: trunk
Choose a base branch
from
Open

Implementing second level classLoading and JSonProvider #4

wants to merge 1 commit into from

Conversation

clebertsuconic
Copy link

I am having issue with Artemis in the following case.

  • a RMI Call is made through our management, and we have no control over the TCCL. We would need to make such controls on our app, and I believe it would be better to be part of the provider, hence this PR.
  • The provider is making excessive classloading operation for any JSON created. it would be better to cache results. Instead of caching it forever I used a SoftReference here.

for more information, this was discussed on a pull request on Artemis at : apache/activemq-artemis#684

@clebertsuconic clebertsuconic mentioned this pull request Jul 29, 2016
Merged
@clebertsuconic
Copy link
Author

Any idea if this can make the codebase? any adjustments to be made?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@clebertsuconic