Bug 1253398 - ClusteredInvoker should reuse HttpInvokers (performance)
Summary: ClusteredInvoker should reuse HttpInvokers (performance)
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: JBoss Fuse Service Works 6
Classification: JBoss
Component: SwitchYard
Version: 6.0.0 GA
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: CR1
: ---
Assignee: tcunning
QA Contact: Pavel Macik
URL:
Whiteboard:
Depends On:
Blocks: 1214695
TreeView+ depends on / blocked
 
Reported: 2015-08-13 15:58 UTC by Rick Wagner
Modified: 2025-02-10 03:48 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2025-02-10 03:48:00 UTC
Type: Support Patch
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker SWITCHYARD-2771 0 Major Resolved ClusteredInvoker should reuse HttpInvokers (performance) 2016-03-30 12:27:22 UTC

Description Rick Wagner 2015-08-13 15:58:56 UTC
ClusteredInvoker needs to reuse HttpInvokers (otherwise we may have performance problems downstream because a new invoker means a new Serializer, a new ObjectMapper within that serializer, etc etc).

https://github.com/errantepiphany/switchyard-core/blob/master/remote/src/main/java/org/switchyard/remote/cluster/ClusteredInvoker.java#L57

David Ward notes :
What could be done to improve performance is...
2:08 Have a WeakHashMap that maps the URL behind ep.getEndpoint() to an HttpInvoker.
2:09 That way we only create new HttpInvokers for new URLs (which doesn't happen much), but since it's a WeakHashMap, the GC can clean it up if endpoint URLs change.

Comment 1 tcunning 2015-10-12 16:23:56 UTC
Backported to 1.x core.

Comment 3 Red Hat Bugzilla 2025-02-10 03:48:00 UTC
This product has been discontinued or is no longer tracked in Red Hat Bugzilla.


Note You need to log in before you can comment on or make changes to this bug.