chore: use errors.New to replace fmt.Errorf with no parameters

[ChengenH]

No description provided.

[p-shahi]

From reading golang docs, it seems like both of these are almost identical under the hood. Is there a good reason to switch? cc @sukunrt or @MarcoPolo

[ChengenH]

From reading golang docs, it seems like both of these are almost identical under the hood. Is there a good reason to switch? cc @sukunrt or @MarcoPolo

errors.New has better performance, although slightly, since it does not involve the overhead of formatting the string. It is also more concise when there are no formatting arguments.

[marten-seemann]

I don’t think performance in the error code path matters at all. This seems like an unsubstantiated performance claim without a benchmark.