GGraziadei commented on PR #8593: URL: https://github.com/apache/storm/pull/8593#issuecomment-4835690399
Thank you both for the review and the insights! I have updated the Javadoc to frame the stream grouping behavior and benchmark results in a more neutral way. Regarding @reiabreu point about FEEDBACK_TICK_STREAM_ID, this actually extends to all control plane traffic, such as metrics ticks and heartbeats. The issue is more generic and definitely non-blocking for this PR. To address this, I have opened a follow-up issue here: #8816 I’ve already included an initial solution with its pros and cons in the description, but I will also start a thread on the dev@ mailing list shortly to get feedback from the community, maybe someone has an even better approach! Merging this now. Thanks again! -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
