Skip to content

MinSafeTS support TiFlash#642

Merged
sticnarf merged 8 commits intotikv:masterfrom
hehechen:staleread
Dec 19, 2022
Merged

MinSafeTS support TiFlash#642
sticnarf merged 8 commits intotikv:masterfrom
hehechen:staleread

Conversation

@hehechen
Copy link
Contributor

Signed-off-by: hehechen awd123456sss@gmail.com
MinSafeTS support TiFlash to support TiFlash stale read.

Signed-off-by: hehechen <awd123456sss@gmail.com>
@hehechen
Copy link
Contributor Author

/cc @sticnarf @Connor1996

@hehechen
Copy link
Contributor Author

/run-all-tests

Copy link
Member

@JmPotato JmPotato left a comment

Choose a reason for hiding this comment

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

Could we add a test for this change also? The rest LGTM.

Copy link
Contributor

@lhy1024 lhy1024 left a comment

Choose a reason for hiding this comment

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

Maybe we need to add some tests

Signed-off-by: hehechen <awd123456sss@gmail.com>
Signed-off-by: hehechen <awd123456sss@gmail.com>
Signed-off-by: hehechen <awd123456sss@gmail.com>
@hehechen
Copy link
Contributor Author

/cc @sticnarf

sticnarf and others added 2 commits December 16, 2022 23:14
Signed-off-by: hehechen <awd123456sss@gmail.com>
@hehechen
Copy link
Contributor Author

/cc @disksing

Copy link
Collaborator

@sticnarf sticnarf left a comment

Choose a reason for hiding this comment

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

LGTM

@sticnarf
Copy link
Collaborator

@hehechen There are race and lint errors.

@hehechen hehechen requested a review from sticnarf December 19, 2022 07:45
@sticnarf
Copy link
Collaborator

There is still a lint error remaining.

You can ignore the integration test failure. It's just flaky.

Signed-off-by: hehechen <awd123456sss@gmail.com>
@hehechen
Copy link
Contributor Author

There is still a lint error remaining.

You can ignore the integration test failure. It's just flaky.

Done. Checked in local machine by golangci-lint run.

@sticnarf sticnarf merged commit fe3536d into tikv:master Dec 19, 2022
@hehechen hehechen deleted the staleread branch December 19, 2022 09:31
hawkingrei pushed a commit to hawkingrei/client-go that referenced this pull request Feb 24, 2023
Signed-off-by: hehechen <awd123456sss@gmail.com>
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.

4 participants