-
Notifications
You must be signed in to change notification settings - Fork 105
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
[to #462] add version constraint for rawkv tests #463
[to #462] add version constraint for rawkv tests #463
Conversation
Signed-off-by: iosmanthus <myosmanthustree@gmail.com>
@@ -21,6 +21,21 @@ | |||
import org.tikv.util.TestUtils; | |||
|
|||
public class BaseRawKVTest { | |||
|
|||
protected String getTiKVVersion() { | |||
String tikvVersion = TestUtils.getEnv("TIKV_VERSION"); |
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.
Who will set the TIKV_VERSION
?
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.
we can get the tikv version from TiKV, see https://github.com/tikv/client-java/blob/master/src/main/java/org/tikv/common/StoreVersion.java.
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.
Cool
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.
Who will set the
TIKV_VERSION
?
By design, like RAWKV_PD_ADDRESSES, it will be set by the test executor.
…-constraint-for-tests
Signed-off-by: iosmanthus <myosmanthustree@gmail.com>
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.
LGTM
Signed-off-by: iosmanthus <myosmanthustree@gmail.com>
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.
LGTM
/merge |
/run-all-tests |
Signed-off-by: iosmanthus myosmanthustree@gmail.com
What problem does this PR solve?
Issue Number: close #462
Problem Description:
Some tests in the master branch are not compatible with older versions of TiKV, we need to add some version constraints to these tests and furthermore, add integration compatibility tests for the master branch.
What is changed and how it works?
Use
Assume
to ignore some tests with version info reading from ENV.Check List for Tests
This PR has been tested by at least one of the following methods: