-
Notifications
You must be signed in to change notification settings - Fork 24.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
Add production warning for pre-release builds #20674
Conversation
This commit adds a usage warning when Elasticsearch is started with a pre-release build.
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 folks are going to read this but we may as well.
@@ -313,6 +313,10 @@ public String toString() { | |||
return sb.toString(); | |||
} | |||
|
|||
public String displayVersion() { |
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 foresee being confused about whether we should use this or toString
. Maybe we should just use toString
in this case?
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.
@nik9000 Version#toString
does not display whether or not the build is a snapshot build; I'm happy to add Javadocs?
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.
Ah! I see the static reference now. I think this method is fairly confusing then? Like, it looks like it is just about the version that you are calling it on but it does stuff with the current build.
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 pushed 0fce68d, I think this will remove the confusion.
This commit moves Version#displayVersion to Node as the method is rather confusing on Version since it relies on a reference to an instance of Version but a static reference to the build.
This commit adds a usage warning when Elasticsearch is started with a pre-release build. Relates #20674
This commit adds a usage warning when Elasticsearch is started with a pre-release build. Relates #20674
Thanks @nik9000. |
This commit adds a usage warning when Elasticsearch is started with a
pre-release build.