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

CAMEL-19102: cache the producer and endpoint locally to avoid hitting the type check scalability issue #9986

Merged
merged 1 commit into from
May 15, 2023
Merged

CAMEL-19102: cache the producer and endpoint locally to avoid hitting the type check scalability issue #9986

merged 1 commit into from
May 15, 2023

Conversation

orpiske
Copy link
Contributor

@orpiske orpiske commented May 3, 2023

This reworks the patch originally added on 907a9d0 so that there is no side effect when handling singleton endpoints

@github-actions
Copy link
Contributor

github-actions bot commented May 3, 2023

🌟 Thank you for your contribution to the Apache Camel project! 🌟

🐫 Maintainers, please note that first-time contributors require manual approval for the GitHub Actions to run.

⚠️ Please note that the changes on this PR may be tested automatically if they change components.

If necessary Apache Camel Committers may access logs and test results in the job summaries!

@orpiske orpiske changed the title CAMEL-19058: cache the producer and endpoint locally to avoid hitting the type check scalability issue" CAMEL-19058: cache the producer and endpoint locally to avoid hitting the type check scalability issue May 3, 2023
@github-actions
Copy link
Contributor

github-actions bot commented May 3, 2023

🚫 There are (likely) no components to be tested in this PR

2 similar comments
@github-actions
Copy link
Contributor

github-actions bot commented May 3, 2023

🚫 There are (likely) no components to be tested in this PR

@github-actions
Copy link
Contributor

github-actions bot commented May 3, 2023

🚫 There are (likely) no components to be tested in this PR

@github-actions
Copy link
Contributor

github-actions bot commented May 3, 2023

Components tested:

Total Tested Failed ❌ Passed ✅
1 1 0 2

@orpiske orpiske marked this pull request as draft May 5, 2023 05:36
@orpiske
Copy link
Contributor Author

orpiske commented May 5, 2023

Folks, I am marking this one as a draft. I am thinking about linking a few related PRs, so it may be easier to look at the whole set of changes together.

@orpiske
Copy link
Contributor Author

orpiske commented May 15, 2023

I am back at looking at this one. The patch is (likely) fine, but I want to review the results one more time.

@orpiske
CAMEL-19058: cache the producer and endpoint locally to avoid hitting…
641b610 
… the type check scalability issue

This reworks the patch originally added on c8d36cb (and later reverted) so that there is no side effect when handling singleton endpoints
@orpiske orpiske changed the title CAMEL-19058: cache the producer and endpoint locally to avoid hitting the type check scalability issue CAMEL-19102: cache the producer and endpoint locally to avoid hitting the type check scalability issue May 15, 2023
@github-actions
Copy link
Contributor

🚫 There are (likely) no components to be tested in this PR

@orpiske
Copy link
Contributor Author

orpiske commented May 15, 2023

chart(1)

Just waiting for the JMH to run and we should be good to go (above vs. M1)

@orpiske orpiske marked this pull request as ready for review May 15, 2023 16:21
@orpiske orpiske merged commit 921ce51 into apache:main May 15, 2023
3 of 4 checks passed
@orpiske orpiske deleted the camel-19102 branch April 10, 2024 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@davsclaus davsclaus davsclaus left review comments

@oscerd oscerd oscerd approved these changes

Assignees
No one assigned
Labels
camel-4 core
Projects
None yet
Milestone
No milestone
3 participants
@orpiske @davsclaus @oscerd