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 edition field for the version information #7821
Conversation
Signed-off-by: zhangjinpeng1987 <zhangjinpeng@pingcap.com>
src/lib.rs
Outdated
@@ -66,15 +66,18 @@ pub mod storage; | |||
/// Returns the tikv version information. | |||
pub fn tikv_version_info() -> String { | |||
let fallback = "Unknown (env var does not exist when building)"; | |||
let edition = "Community"; |
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.
How to define values other than community?
Signed-off-by: zhangjinpeng1987 <zhangjinpeng@pingcap.com>
…into community-edition
Signed-off-by: zhangjinpeng1987 <zhangjinpeng@pingcap.com>
It's better to use the |
I think it's better to keep the existing style. |
Your auto merge job has been accepted, waiting for:
|
/run-all-tests |
/run-all-tests |
@zhangjinpeng1987 merge failed. |
/merge |
/run-all-tests |
Signed-off-by: zhangjinpeng1987 <zhangjinpeng@pingcap.com>
cherry pick to release-4.0 in PR #7831 |
Signed-off-by: zhangjinpeng1987 zhangjinpeng@pingcap.com
What problem does this PR solve?
Add the edition field for the version information.
What is changed and how it works?
Add the edition field for the version information.
Related changes
Check List
Tests
Side effects
Release note