[
https://issues.apache.org/jira/browse/STORM-2032?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15416361#comment-15416361
]
ASF GitHub Bot commented on STORM-2032:
---------------------------------------
Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/storm/pull/1618#discussion_r74361223
--- Diff:
storm-core/src/jvm/org/apache/storm/messaging/netty/StormClientHandler.java ---
@@ -60,7 +60,6 @@ public void messageReceived(ChannelHandlerContext ctx,
MessageEvent event) {
//This should be the metrics, and there should only be one
of them
--- End diff --
If we want to remove warning message because this is not a part of design,
how about modifying this comment as well?
Maybe like this : `This should be the metrics, and it always picks the last
one when more than one metrics messages are available.`
> "not fast enough" metrics WARN message in netty client can be misinterpreted
> ----------------------------------------------------------------------------
>
> Key: STORM-2032
> URL: https://issues.apache.org/jira/browse/STORM-2032
> Project: Apache Storm
> Issue Type: Bug
> Components: storm-core
> Affects Versions: 2.0.0
> Reporter: Derek Dagit
> Assignee: Derek Dagit
> Priority: Minor
>
> Example:
> bq. WARN Messages are not being delivered fast enough, got 3 metrics messages
> at once
> It appears to some users to be a good signal for monitoring/alerting, but
> really this is not part of the design.
> We should remove it, change it, or lower the log level.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
