Skip to content
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

Explain EatWhatYouKill naming #6342

Merged
merged 2 commits into from
Jun 4, 2021

Conversation

gregw
Copy link
Contributor

@gregw gregw commented Jun 1, 2021

Some concern has been raised at the EatWhatYouKill strategy name (OK @sbordet you get I told you so rights).
This PR updates the javadoc to be clear about the reason for using that name in that the strategy is named after
a hunting proverb in the sense
that one should only kill to eat. The use of this phrase is not an endorsement of hunting nor killing of wildlife for
food or sport.

Signed-off-by: Greg Wilkins <[email protected]>
@gregw gregw requested a review from sbordet June 1, 2021 23:51
Signed-off-by: Greg Wilkins <[email protected]>
@sbordet
Copy link
Contributor

sbordet commented Jun 3, 2021

@gregw at the end it's a dynamic ExecutionStrategy that changes its mode based on TryExecutor.tryExecute().

So perhaps it's time to call it DynamicExecutionStrategy and deprecate EatWhatYouKill?

Copy link
Contributor

@sbordet sbordet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Additional text is good.
However, it's probably time to change the name as I specify in the comments.

@gregw
Copy link
Contributor Author

gregw commented Jun 3, 2021

@sbordet I think you are right - the EWYK is now only one of several strategies that can be applied, and then when compared with PC & PEC it is clearer to describe it as EPC.

I'll do the rename, but only in 10 and leave the EWYK as a deprecated superclass of it.

@sbordet
Copy link
Contributor

sbordet commented Jun 3, 2021

@gregw EWYK should be a deprecated subclass of DynamicExecutionStrategy.

@sbordet sbordet merged commit 40a9fa7 into jetty-9.4.x Jun 4, 2021
@sbordet sbordet deleted the jetty-9.4.x-eat-what-you-kill-name branch June 4, 2021 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants