[fix #67] fix golangci-lint failures for TiKV BR

[zz-jason]

Signed-off-by: Jian Zhang [email protected]

What problem does this PR solve?

Issue Number: [fix #67]

Problem Description: golangci-lint failure is the last breaking check in CI

What is changed and how does it work?

there are lots of lint failures, I fixed them in two ways:

• goimports failures are fixed by goimports -e -w .
• others are fixed manually

Code changes

• Has exported function/method change

Check List for Tests

This PR has been tested by at least one of the following methods:

• Unit test
• Integration test

Side effects

• No side effects

Related changes

• No related changes

[zz-jason]

/verify

[zz-jason]

golangci-lint faild in verify is caused by:

[](https://ci.pingcap.net/blue/organizations/jenkins/lint-check/detail/lint-check/3563/pipeline/#step-55-log-156)[](https://ci.pingcap.net/blue/organizations/jenkins/lint-check/detail/lint-check/3563/pipeline/#step-55-log-157)[2022-03-22T02:04:32.574Z] level=info msg="[loader] Go packages loading at mode 575 (name|compiled_files|exports_file|files|imports|deps|types_sizes) took 1m0.947348183s"
[2022-03-22T02:04:32.574Z] level=error msg="Running error: context loading failed: failed to load packages: timed out to load packages: context deadline exceeded"
[2022-03-22T02:04:32.574Z] level=error msg="Timeout exceeded: try increasing it by passing --timeout option"

--timeout parameter of golangci-lint is default to 1m, I changed it to 10m in the CI / br-golangci-lint workflow.

[zeminzhou]

LGTM~

[pingyu]

LGTM