Markers
Table of Contents
Logging.php
| Type | Line | Description |
|---|---|---|
| TODO | 16 | this should be `@internal`, but deptrac is not happy with that. |
ExtensionHookManager.php
| Type | Line | Description |
|---|---|---|
| TODO | 37 | Add an option flag to ext-opentelemetry `hook` that configures whether return values should be used? |
AggregationResolverDefault.php
| Type | Line | Description |
|---|---|---|
| TODO | 25 | Implement proper aggregation providers (default, drop, explicit_bucket_histogram, last_value, sum) to handle advisory |
OpenTelemetrySdk.php
| Type | Line | Description |
|---|---|---|
| TODO | 188 | Add unit criteria |
| TODO | 211 | Add support for aggregation providers in views to allow usage of advisory |
| TODO | 239 | Allow injecting log record attributes factory |
ContentTypes.php
| Type | Line | Description |
|---|---|---|
| TODO | 10 | enum (php >= 8.1) |
Temporality.php
| Type | Line | Description |
|---|---|---|
| TODO | 13 | convert to enum (php >= 8.1) |
MeterProviderFactory.php
| Type | Line | Description |
|---|---|---|
| TODO | 32 | https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/metrics/sdk_exporters/otlp.md#general - "The exporter MUST configure the default aggregation on the basis of instrument kind using the OTEL_EXPORTER_OTLP_METRICS_DEFAULT_HISTOGRAM_AGGREGATION variable as described below if it is implemented." |
| TODO | 38 | "The SDK MAY accept a comma-separated list to enable setting multiple exporters" |
CriteriaViewRegistry.php
| Type | Line | Description |
|---|---|---|
| TODO | 28 | is null the best return type here? what about empty array or exception? |
Registry.php
| Type | Line | Description |
|---|---|---|
| TODO | 21 | [breaking] deprecate this mechanism of setting up components, in favor of using SPI. |
ResourceInfo.php
| Type | Line | Description |
|---|---|---|
| TODO | 64 | can we optimize this to avoid re-validating the attributes on merge? |
TraceIdRatioBasedSampler.php
| Type | Line | Description |
|---|---|---|
| TODO | 49 | Add config to adjust which spans get sampled (only default from specification is implemented) |
SimpleSpanProcessorTest.php
| Type | Line | Description |
|---|---|---|
| TODO | 121 | Add test to ensure exporter is retried on failure. |