You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by "orpiske (via GitHub)" <gi...@apache.org> on 2023/05/22 13:17:38 UTC

[GitHub] [camel-performance-tests] orpiske opened a new pull request, #118: EndpointRegistry tests

orpiske opened a new pull request, #118:
URL: https://github.com/apache/camel-performance-tests/pull/118

   (no comment)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel-performance-tests] davsclaus commented on pull request #118: EndpointRegistry tests

Posted by "davsclaus (via GitHub)" <gi...@apache.org>.
davsclaus commented on PR #118:
URL: https://github.com/apache/camel-performance-tests/pull/118#issuecomment-1557240296

   It may also be interresting to test with camel-caffeine-lrucache on the classpath to use caffeeine as thread-safe cache. Then we can compare this with #10157
   
   Also #10157 can potentially be optimized to only sync on certain situations.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel-performance-tests] orpiske merged pull request #118: EndpointRegistry tests

Posted by "orpiske (via GitHub)" <gi...@apache.org>.
orpiske merged PR #118:
URL: https://github.com/apache/camel-performance-tests/pull/118


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel-performance-tests] orpiske commented on pull request #118: EndpointRegistry tests

Posted by "orpiske (via GitHub)" <gi...@apache.org>.
orpiske commented on PR #118:
URL: https://github.com/apache/camel-performance-tests/pull/118#issuecomment-1558550047

   > LGTM, I think we should add the license everywhere in this repo.
   
   Indeed! I created a ticket so I don't forget to fix these ... we have quite a couple ones missing the license information. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel-performance-tests] orpiske commented on pull request #118: EndpointRegistry tests

Posted by "orpiske (via GitHub)" <gi...@apache.org>.
orpiske commented on PR #118:
URL: https://github.com/apache/camel-performance-tests/pull/118#issuecomment-1557221131

   @davsclaus @rhuan080 Guys, do you mind sharing your thoughts about this?
   
   Claus had suggested, on the discussion on apache/camel#9972, that the LRUCache was an interesting test scenario. Here I am providing a set of tests that should cover a few of the areas that I believe could be hit by the LRUCache changes. 
   
   There are 4 new tests added and they should cover the following aspects: 
   
   * `EndpointCreationTest`: compute the accumulated time of all the operations required to create a given number of routes (500, in this test).
   *  `EndpointResolveTest`: compute the accumulated time of all the operations required to resolve the same route over and over (this is a rather "macro" operation, so it's not something that would be inlined) 
   * EndpointRegistryTest: tests specific operations of the Endpoint registry in single-thread scenarios.
   * EndpointRegistryScalabilityTest: tests specific operations of the Endpoint registry in multi-thread scenarios.
   
   I am currently running the baselines for these (on 4.0 `main` and 4.0 M1). After that, I'll run it with the changes on apache/camel#10157 to verify the impacts.
   
   Please let me know if you notice something is missing or wrong.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org