-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[cherry-pick-2.1] Fix event listener issue #4132
[cherry-pick-2.1] Fix event listener issue #4132
Conversation
* Check status of CompactionJobInfo Signed-off-by: DorianZheng <xingzhengde72@gmail.com> (cherry picked from commit 22957b7) Signed-off-by: DorianZheng <xingzhengde72@gmail.com>
Signed-off-by: DorianZheng <xingzhengde72@gmail.com>
CI failed seem now master and release use different rust version. |
@DorianZheng We should add a release-2.1 branch without Titan related code in pingcap/rust-rocksdb. |
@siddontang yes they are on different versions now. What do you think we should do about it? How long is it between now and the next release? |
Ping @DorianZheng |
/run-all-tests |
This comment has been minimized.
This comment has been minimized.
Signed-off-by: DorianZheng <xingzhengde72@gmail.com>
173b926
to
2ff9478
Compare
@overvenus I will create a release-2.1 branch for rust-rocksdb, so never mind the previous CI failed |
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
/run-integration-tests tidb=release-2.1 pd=release-2.1 tidb-test=release-2.1 |
Signed-off-by: DorianZheng <xingzhengde72@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
What have you changed? (mandatory)
Check status of CompactionJobInfo at first.
What are the type of the changes? (mandatory)
How has this PR been tested? (mandatory)
CI
Does this PR affect documentation (docs) update? (mandatory)
No
Does this PR affect tidb-ansible update? (mandatory)
No
Refer to a related PR or issue link (optional)
Benchmark result if necessary (optional)
Add a few positive/negative examples (optional)