chore: Use error equals instead of strings.Contains

[juchaosong]

Checklist:

• Either (a) I've created an enhancement proposal and discussed it with the community, (b) this is a bug fix, or (c) this is a chore.
• The title of the PR is (a) conventional, (b) states what changed, and (c) suffixes the related issues number. E.g. "fix(controller): Updates such and such. Fixes #1234".
• I have written unit and/or e2e tests for my change. PRs without these are unlike to be merged.
• Optional. I've added My organization is added to the USERS.md.
• I've signed the CLA and required builds are green.

[CLAassistant]

)
All committers have signed the CLA.

[alexec]

Can I please ask what this fixes?

[juchaosong]

I think it's more readable.
I do the same thing like argo archive. When I read argo source code, I wonder why it uses strings.Contains(err.Error(), "no more rows") to determine if there is no record that meets the conditions. And I find in db.v3 used in argo, when there's no records, it will raise db.ErrNoMoreRows. So in my code, I use db.ErrNoMoreRows instead of strings.Contains(). I think it's more readable for user who use and develop argo.

[codecov]

Codecov Report

Merging #2636 into master will not change coverage by %.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master    #2636   +/-   ##
=======================================
  Coverage   11.16%   11.16%           
=======================================
  Files          83       83           
  Lines       32673    32673           
=======================================
  Hits         3649     3649           
  Misses      28525    28525           
  Partials      499      499           

Impacted Files	Coverage Δ
persist/sqldb/workflow_archive.go	0.00% <0.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8c29e05...57931b5. Read the comment docs.