runtime icon indicating copy to clipboard operation
runtime copied to clipboard

Replace zap with slog or logr

Open shanduur opened this issue 1 year ago • 1 comments

Zap is opinionated. It would be a great idea to support a generic logging interface (e.g. with github.com/go-logr/logr or log/slog) instead forcing users to use Zap.

shanduur avatar Apr 10 '24 11:04 shanduur

The main benefit of zap function and methods is that they allocate as little as possible (zerolog is even better in that regard, but has some additional quirks). slog is not so efficient, and we really need performant logs.

I know zap is not something everyone wants, but COSI runtime already makes several opinionated decisions, so there that.

DmitriyMV avatar Apr 10 '24 17:04 DmitriyMV

Nothing more to add, closing.

DmitriyMV avatar Jan 15 '25 16:01 DmitriyMV