-
Notifications
You must be signed in to change notification settings - Fork 3.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HBASE-25547 (addendum): Roll ExecutorType into ExecutorConfig #2996
Conversation
I missed addressing this in the original PR, mind taking a look at the addendum? |
} | ||
startExecutorService(config.setName(name)); | ||
} | ||
|
||
/** | ||
* Initialize the executor lazily, Note if an executor need to be initialized lazily, then all |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure I understood this comment. In what sense this is lazier compared to startExecutorService()? Wanted to get rid of this but checking here first.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't get this either, method name also has lazy word.
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
} | ||
startExecutorService(config.setName(name)); | ||
} | ||
|
||
/** | ||
* Initialize the executor lazily, Note if an executor need to be initialized lazily, then all |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't get this either, method name also has lazy word.
9a2ed4a
to
4c822d7
Compare
No description provided.