You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@storm.apache.org by GitBox <gi...@apache.org> on 2020/06/24 18:52:36 UTC

[GitHub] [storm] RuiLi8080 commented on pull request #3289: [STORM-3654] remove executor id from JCQueue receive-queue name

RuiLi8080 commented on pull request #3289:
URL: https://github.com/apache/storm/pull/3289#issuecomment-649003061


   @agresch Thanks for the approval. Talked to @Ethanlm offline, he suggested to use `metricNamePrefix` instead of `owner` which seems to be more confusing. I agreed since essentially we just want to prevent including executor id in metric name. This way we can still keep the old way of queue name for logging.
   Just pushed new changes for review.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org