[GitHub] [storm] Ethanlm commented on a change in pull request #3411: STORM-3793 add metric to track backpressure status for tasks

2021-09-22 Thread GitBox
Ethanlm commented on a change in pull request #3411: URL: https://github.com/apache/storm/pull/3411#discussion_r714037922 ## File path: storm-client/src/jvm/org/apache/storm/daemon/worker/BackPressureTracker.java ## @@ -118,8 +118,26 @@ public void

[GitHub] [storm] agresch commented on a change in pull request #3411: STORM-3793 add metric to track backpressure status for tasks

2021-09-22 Thread GitBox
agresch commented on a change in pull request #3411: URL: https://github.com/apache/storm/pull/3411#discussion_r714038485 ## File path: storm-client/src/jvm/org/apache/storm/daemon/worker/BackPressureTracker.java ## @@ -118,8 +118,26 @@ public void

[GitHub] [storm] agresch commented on a change in pull request #3411: STORM-3793 add metric to track backpressure status for tasks

2021-09-22 Thread GitBox
agresch commented on a change in pull request #3411: URL: https://github.com/apache/storm/pull/3411#discussion_r714050012 ## File path: storm-client/src/jvm/org/apache/storm/daemon/worker/BackPressureTracker.java ## @@ -118,8 +118,26 @@ public void

[GitHub] [storm] agresch merged pull request #3411: STORM-3793 add metric to track backpressure status for tasks

2021-09-22 Thread GitBox
agresch merged pull request #3411: URL: https://github.com/apache/storm/pull/3411 -- 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:

[GitHub] [storm] Ethanlm commented on a change in pull request #3411: STORM-3793 add metric to track backpressure status for tasks

2021-09-22 Thread GitBox
Ethanlm commented on a change in pull request #3411: URL: https://github.com/apache/storm/pull/3411#discussion_r714057775 ## File path: storm-client/src/jvm/org/apache/storm/daemon/worker/BackPressureTracker.java ## @@ -118,8 +118,26 @@ public void

[GitHub] [storm] Ethanlm commented on pull request #3412: [STORM-3785] Follow up: fix the checkstyle issue

2021-09-22 Thread GitBox
Ethanlm commented on pull request #3412: URL: https://github.com/apache/storm/pull/3412#issuecomment-925238183 introduced by https://github.com/apache/storm/pull/3405 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [storm] Ethanlm merged pull request #3405: [STORM-3785] Stop converting V2 rate metrics when adding to V1

2021-09-22 Thread GitBox
Ethanlm merged pull request #3405: URL: https://github.com/apache/storm/pull/3405 -- 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:

[GitHub] [storm] Ethanlm merged pull request #3412: [STORM-3785] Follow up: fix the checkstyle issue

2021-09-22 Thread GitBox
Ethanlm merged pull request #3412: URL: https://github.com/apache/storm/pull/3412 -- 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:

[GitHub] [storm] RuiLi8080 opened a new pull request #3412: [MINOR] checkstyle

2021-09-22 Thread GitBox
RuiLi8080 opened a new pull request #3412: URL: https://github.com/apache/storm/pull/3412 ## What is the purpose of the change *(Explain why we should have this change)* ## How was the change tested *(Explain what tests did you do to verify the code change)* -- This

[GitHub] [storm] Ethanlm commented on pull request #3395: [STORM-3767] fix NPE in getComponentPendingProfileActions

2021-09-22 Thread GitBox
Ethanlm commented on pull request #3395: URL: https://github.com/apache/storm/pull/3395#issuecomment-925182399 merged in master branch. Will cherry-pick it to other branches. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub