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

[Spike] Investigate Increased Overhead with High Volume Token Linking #1988

Open
deleonenriqueta opened this issue Jul 22, 2024 · 3 comments
Labels
spike Research spike. Technical solution needing discovery

Comments

@deleonenriqueta
Copy link
Contributor

Description

When using a Reactor Stack and processing a high volume of requests, we've observed an increase in application overhead (CPU usage and latency).

Acceptance Criteria

Research spike. Investigate whether anything can feasibly be done to optimize performance and reduce this overhead, specifically focusing on token linking.

Additional context

Review the GTSE that references this behavior to gather more information.

@deleonenriqueta deleonenriqueta added the spike Research spike. Technical solution needing discovery label Jul 22, 2024
@workato-integration
Copy link

@meiao
Copy link
Contributor

meiao commented Aug 27, 2024

Another customer noticed some increase in overhead. Given the versions the customer was using, we advised them to disable some classes from being weaved and it decreased the overhead.

The configuration for the exclusions is:

class_transformer:
  excludes: ^reactor/core/publisher/MonoSubscribeOn, ^reactor/core/publisher/MonoSubscribeOn$SubscribeOnSubscriber, ^reactor/core/publisher/FluxMapFuseable, ^reactor/core/publisher/FluxMapFuseable$MapFuseableSubscriber

@dvilaverde
Copy link

The $ character in the "excludes" stanza needs to be escaped, so use the following to exclude those classes:

class_transformer:
    excludes: ^reactor/core/publisher/MonoSubscribeOn, ^reactor/core/publisher/MonoSubscribeOn\$SubscribeOnSubscriber, ^reactor/core/publisher/FluxMapFuseable, ^reactor/core/publisher/FluxMapFuseable\$MapFuseableSubscriber

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spike Research spike. Technical solution needing discovery
Projects
None yet
Development

No branches or pull requests

3 participants