csudharsanan commented on code in PR #2696:
URL: https://github.com/apache/helix/pull/2696#discussion_r1387195493
##########
helix-core/src/main/java/org/apache/helix/messaging/handling/HelixTaskExecutor.java:
##########
@@ -276,7 +276,7 @@ private void startMonitorThread() {
/** Dedicated Thread pool can be provided in configuration or by client.
* This method is to check it and update the thread pool if necessary.
*/
- private void updateStateTransitionMessageThreadPool(Message message,
HelixManager manager) {
+ public void updateStateTransitionMessageThreadPool(Message message,
HelixManager manager) {
Review Comment:
Had to move the TestClass from **org.apache.helix** Package to
**org.apache.helix.messaging.handling** Package to use annotations. Since there
was another test class with the same name **TestHelixExecutor** in the package
: org.apache.helix.messaging.handling, I ended up renaming this one to
**TestHelixExecutor2**.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
