tarantool / metrics

Metric collection library for Tarantool
MIT License
39 stars 23 forks source link

internal: remove circular dependencies #434

Closed DifferentialOrange closed 1 year ago

DifferentialOrange commented 1 year ago

This PR removes existing circular dependencies in module, as well as remaining requires inside the functions.

Before:

image

After:

image

(Still seems confusing, but at least more hierarchic.)

It is a prerequirement to metrics.cfg{} reload support.

I didn't forget about

Closes #433, part of tarantool/tarantool#7725

yngvar-antonsson commented 1 year ago

Is this schema handmade or generated?

DifferentialOrange commented 1 year ago

Is this schema handmade or generated?

I didn't even though that it is possible to automatize the process, so it's hand-made with app.diagrams.net . (I can share the sources if you need them.)