Make more linters really happy

[hi-rustin]

Description

Currently, we use "golangci-lint" to execute some linters, but due to some reasons, the entire project code may not really pass some necessary linters.
This issue is for everyone to work together to pass some necessary linters for this project. The specific parameters that need to be supported are shown in the task list below.

Related Information

• golangci-lint

Task List

• --enable=unconvert(*: Make unconvert linter happy and locking golangci-lint version #2172)
• --enable=lll
• --enable=unparam(*: enable unparam static check #2368)
• --enable=gosec
• --enable=funlen
• --enable=gocyclo

If there are more necessary linter you would like to enable, please feel free to comment.

[hi-rustin]

cc: @amyangfei @leoppro @overvenus

While reading the code, I noticed that we have some unnecessary type conversion code, so I would like to turn on the unconvert plugin to improve the code quality. What do you think?

[hi-rustin]

/assign

I am working on this.

[hi-rustin]

I found in the process of reading the code that a lot of our code is too long for one screen to read easily. So I suggest we use lint to control the length of each line of code.

[hi-rustin]

There are many parameters in our code that are not used. We can use unparam to detect them!

Original idea from #2309 (comment). cc: @amyangfei