search

yiisoft / di

PSR-11 compatible DI container and injector
https://www.yiiframework.com
BSD 3-Clause "New" or "Revised" License
182 stars 46 forks source link

Collect stats from yii-demo #206

Open samdark opened 3 years ago

samdark commented 3 years ago
  1. How many services are requested per route?
  2. How many aren't defined in container and automatically resolved?
  3. How many sub-dependencies are per service?
yiiliveext commented 3 years ago
  1. How many services are requested per route?
  2. How many aren't defined in container and automatically resolved?
  3. How many sub-dependencies are per service?

This easy imlements with yii-debug.

samdark commented 3 years ago

We need to get these numbers once to better understand what may affect performance and what may not. That's one time thing.

alamagus commented 1 year ago

Any thoughts on having built-in support for opentelemetry, like, yiisoft/yii-otel? https://github.com/open-telemetry/opentelemetry-php (it's not production ready yet though) https://groups.google.com/g/php-fig/c/6Nm0ZQ1_hJA/m/Fxp6EPsODgAJ

samdark commented 1 year ago

@alamagus I don't think we are going to support it. Instead, I'd rely on community solutions for it.

xepozz commented 1 year ago

Any thoughts on having built-in support for opentelemetry, like, yiisoft/yii-otel? https://github.com/open-telemetry/opentelemetry-php (it's not production ready yet though) https://groups.google.com/g/php-fig/c/6Nm0ZQ1_hJA/m/Fxp6EPsODgAJ

@alamagus Thanks for the information, but I think this issue is not about telemetry in general, but about some benchmarks in CI. If you want to add Open Telemetry integration it would be nice to have as separated package that could be included into users apps. Both yiisoft/demo and yiisoft/demo-api repos may be lab rats for these changes. Would you like to implement it?