gomod: fix MinSafeTS might be set to MaxUint64 permanently (#47469)

[ti-chi-bot]

This is an automated cherry-pick of #47469

Signed-off-by: husharp [email protected]<!--

Thank you for contributing to TiDB!

PR Title Format:

1. pkg [, pkg2, pkg3]: what's changed
2. *: what's changed

-->

What problem does this PR solve?

Issue Number: close #47468

Problem Summary:

What is changed and how it works?

The core question is getting safe ts in client-go

• we introduce PD API to not execute go func which for KV request, resulting in not updating safeTSMap.
  • updateMinSafeTS relies on safeTSMap which makes sense(because actually, we can call updateMinSafeTS to kvReuqestUpdater[to indicate func base]).
• And we need to update minsafeTS to make sure when API fails we can fall back to the original way which is by kv request.
• But the core problem is: updateMinSafeTS will return maxUnit64 when the first kv request returns 0 and then although PD API returns correctly[maybe kv is not initialized], TS can not change maxUnit64.
  • to resolve this question, we need to regard maxUnit64 as 0 which means there is an initial state.

Check List

Tests

• Manual test (add detailed scripts or steps below)

Release note

Please refer to Release Notes Language Style Guide to write a quality release note.

None

[HuSharp]

@JmPotato @nolouch PTAL, thx!

[HuSharp]

client-go pr tikv/client-go#999

[HuSharp]

[ti-chi-bot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: JmPotato
Once this PR has been reviewed and has the lgtm label, please assign buchuitoudegou for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[ti-chi-bot]

[LGTM Timeline notifier]

Timeline:

• 2023-11-21 07:57:31.662749007 +0000 UTC m=+304680.327975199: ☑️ agreed by JmPotato.

[codecov]

Codecov Report

❗ No coverage uploaded for pull request base (release-6.5@6907b47). Click here to learn what that means.
The diff coverage is n/a.

Additional details and impacted files

@@               Coverage Diff                @@
##             release-6.5     #47554   +/-   ##
================================================
  Coverage               ?   73.6594%           
================================================
  Files                  ?       1085           
  Lines                  ?     348930           
  Branches               ?          0           
================================================
  Hits                   ?     257020           
  Misses                 ?      75440           
  Partials               ?      16470           

[ti-chi-bot]

PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[HuSharp]

updated by https://github.com/pingcap/tidb/pull/48768/files#diff-33ef32bf6c23acb95f5902d7097b7a1d5128ca061167ec0716715b0b9eeaa5f6R93

now can close this pr

[tangenta]

6.5 updated by #48768 (files)