Skip to content

Add new rpc for mvcc get#146

Merged
gengliqi merged 1 commit intotikv:masterfrom
hongyunyan:hongyunyan_mvcc
Jun 14, 2023
Merged

Add new rpc for mvcc get#146
gengliqi merged 1 commit intotikv:masterfrom
hongyunyan:hongyunyan_mvcc

Conversation

@hongyunyan
Copy link
Contributor

No description provided.

@hongyunyan hongyunyan changed the title [WIP]add new rpc for mvcc get Add new rpc for mvcc get Jun 9, 2023
Comment on lines +42 to +47
catch (Exception & e)
{
bo.backoff(boRegionMiss, e);
continue;
}
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we need to drop the region cache of the region before continue

Copy link
Contributor

@JaySon-Huang JaySon-Huang Jun 14, 2023

Choose a reason for hiding this comment

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

the region cache refresh is done inside sendReqToRegion, do not need to refresh it here

Copy link
Contributor

@JaySon-Huang JaySon-Huang left a comment

Choose a reason for hiding this comment

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

LGTM

@hongyunyan
Copy link
Contributor Author

/rebuild

@gengliqi
Copy link
Member

/test

Signed-off-by: hongyunyan <649330952@qq.com>
@hongyunyan
Copy link
Contributor Author

/test

@hongyunyan
Copy link
Contributor Author

/run-idc-jenkins-ci/test

@hongyunyan
Copy link
Contributor Author

/rerun

@gengliqi
Copy link
Member

/retest

@gengliqi
Copy link
Member

/test

@gengliqi
Copy link
Member

/run-all-tests

@gengliqi gengliqi merged commit 3d30868 into tikv:master Jun 14, 2023
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.

3 participants