-
Notifications
You must be signed in to change notification settings - Fork 217
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
Allow peer forwarder to skip sending events to remote peer #3996
Comments
I like the overall proposal. A few things I'd suggest changing:
What is the |
I took another look. To disable peer-forwarding, you set the following in the
|
Completed by #4004 |
2 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is your feature request related to a problem? Please describe.
Peer Forwarder sends some events to remote peer based on the hash function. If the inner processor of Peer forwarder has "when" condition, it is possible an event is forwarded to remote peer first and then get dropped because when condition evaluates to false. This is very sub-optimal. Also in some cases an option to force local aggregation may be needed.
Describe the solution you'd like
Add a new API to
RequiresPeerForwarding.java
some thing likeThis will allow innerProcessor to evaluate
when
condition and also check iflocalOnly
option is configured.For example, aggregate processor could implement the new API as follows
Describe alternatives you've considered (Optional)
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered: