client-go Refine the flashback error handling

Signed-off-by: JmPotato [email protected]

Since the flashback state check is before other checks like EpochNotMatch, so we need to invalidate the region cache also.

Oct 14 '22 09:10 JmPotato

/cc @Defined2014 @sticnarf @Connor1996

Oct 14 '22 09:10 JmPotato

It looks there is somethong wrong with integration test. Could you check why it is failing?

Oct 17 '22 08:10 disksing

Signed-off-by: JmPotato [email protected]

Since the flashback state check is before other checks like EpochNotMatch, so we need to invalidate the region cache also.

Why not move the order of the check after EpochNotMatch?

Oct 17 '22 08:10 Connor1996

Signed-off-by: JmPotato [email protected] Since the flashback state check is before other checks like EpochNotMatch, so we need to invalidate the region cache also.

Why not move the order of the check after EpochNotMatch?

Good idea. I proposed a PR to do this, PTAL. https://github.com/tikv/tikv/pull/13618

Oct 17 '22 08:10 JmPotato

It looks there is somethong wrong with integration test. Could you check why it is failing?

I think it's the same problem as https://github.com/tikv/client-go/issues/449.

Oct 17 '22 08:10 JmPotato