diff --git a/CHANGELOG.md b/CHANGELOG.md index 9ce99e6f3a8..6f1b48d8b5b 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -5,6 +5,7 @@ ### 🚩 Deprecations 🚩 - Deprecate `p[metric|log|trace]otlp.RegiserServer` in favor of `p[metric|log|trace]otlp.RegiserGRPCServer` (#6180) +- Deprecate `pcommon.NewValueString` in favor of `pcommon.NewValueStr` (#6209) ### 💡 Enhancements 💡 @@ -36,22 +37,25 @@ ### 🚩 Deprecations 🚩 -- Deprecate `go.opentelemetry.io/collector/service/featuregate` in favor of `go.opentelemetry.io/collector/featuregate`. (#6094) +- Deprecate `go.opentelemetry.io/collector/service/featuregate` in favor of `go.opentelemetry.io/collector/featuregate` + . (#6094) - Deprecate `pmetric.OptionalType`, unused enum type. (#6096) - Deprecate `ptrace.Span[Link]?.TraceStateStruct` in favor of `ptrace.Span[Link]?.TraceState` (#6085) - Deprecate `pcommon.NewValueBytesEmpty` in favor of `pcommon.NewValueBytes` that now has the same signature. (#6105) - Deprecate `pmetric.MetricDataType` and related constants in favor of `pmetric.MetricType`. (#6127) - Deprecate `pmetric.Metric.DataType()` in favor of `pmetric.Metric.Type()`. (#6127) -- Deprecate `pmetric.NumberDataPoint.[Set]?[Int|Double]Val()` in favor of `pmetric.NumberDataPoint.[Set]?[Int|Double]Value()`. (#6134) +- Deprecate `pmetric.NumberDataPoint.[Set]?[Int|Double]Val()` in favor + of `pmetric.NumberDataPoint.[Set]?[Int|Double]Value()`. (#6134) - Deprecate `pmetric.Exemplar.[Set]?[Int|Double]Val()` in favor of `pmetric.Exemplar.[Set]?[Int|Double]Value()`. (#6134) - Deprecate `p[metric|log|trace]otlp.[Client|Server]` in favor of `p[metric|log|trace]otlp.GRPC[Client|Server]` (#6165) - Deprecate pdata Clone methods in favor of CopyTo for consistency with other pdata structs (#6164) - `pmetric.Metrics.Clone` is deprecated in favor of `pmetric.Metrics.CopyTo` - `ptrace.Traces.Clone` is deprecated in favor of `pmetric.Traces.CopyTo` - `plog.Logs.Clone` is deprecated in favor of `plogs.Logs.CopyTo` -- Rename all `pcommon.Value` getter/setter methods by removing `Val` suffix. (#6092) +- Rename all `pcommon.Value` getter/setter methods by removing `Val` suffix. (#6092) - Old methods with `Val` suffix are deprecated. - - `StringVal` and `SetStringVal` are deprecated in favor of `Str` and `SetStr` to avoid implementing `fmt.Stringer` interface. + - `StringVal` and `SetStringVal` are deprecated in favor of `Str` and `SetStr` to avoid implementing `fmt.Stringer` + interface. - Therefore, `ValueTypeString` is deprecated in favour of `ValueTypeStr` for consistency. ### 💡 Enhancements 💡 @@ -59,7 +63,8 @@ - Add AppendEmpty and EnsureCapacity method to primitive pdata slices (#6060) - Expose `AsRaw` and `FromRaw` `pcommon.Value` methods (#6090) - Convert `ValueTypeBytes` attributes in logging exporter (#6153) -- Updated how `telemetryInitializer` is created so it's instanced per Collector instance rather than global to the process (#6138) +- Updated how `telemetryInitializer` is created so it's instanced per Collector instance rather than global to the + process (#6138) ## v0.60.0 Beta @@ -86,7 +91,8 @@ - Deprecate `pcommon.New[Trace|Span]ID` in favor direct conversion. (#6008) - Deprecate `MetricDataPointFlagsImmutable` type. (#6017) - Deprecate `*DataPoint.[Set]FlagsImmutable()` funcs in favor of `*DataPoint.[Set]Flags()`. (#6017) -- Deprecate `LogRecord.FlagsStruct()` and `LogRecord.SetFlagsStruct()` in favor of `LogRecord.Flags()` and `LogRecord.SetFlags()`. (#6007) +- Deprecate `LogRecord.FlagsStruct()` and `LogRecord.SetFlagsStruct()` in favor of `LogRecord.Flags()` + and `LogRecord.SetFlags()`. (#6007) - Deprecate `config.Unmarshallable` in favor of `confmap.Unmarshaler`. (#6031) - Primitive slice wrapper are now mutable (#5971): - `pcommon.ImmutableByteSlice` is deprecated in favor of `pcommon.ByteSlice` @@ -108,7 +114,8 @@ ### 💡 Enhancements 💡 - Add `skip-get-modules` builder flag to support isolated environment executions (#6009) - - Skip unnecessary Go binary path validation when the builder is used with `skip-compilation` and `skip-get-modules` flags (#6026) + - Skip unnecessary Go binary path validation when the builder is used with `skip-compilation` and `skip-get-modules` + flags (#6026) - Make the otlpreceiver support to use jsoniter to unmarshal JSON payloads. (#6040) - Add mapstructure hook function for confmap.Unmarshaler interface (#6029) - Add CopyTo and MoveTo methods to primitive slices (#6044) @@ -136,16 +143,22 @@ - Remove `pmetric.ExponentialHistogramDataPoint.FlagsStruct()` - Remove `pmetric.SummaryDataPoint.FlagsStruct()` - Remove deprecated settings from `obsreport`, `ProcessorSettings.Level` and `ExporterSettings.Level` (#5918) -- Replace `processorhelper.New[Traces|Metrics|Logs]Exporter` with `processorhelper.New[Traces|Metrics|Logs]ProcessorWithCreateSettings` definition (#5915) -- Replace `exporterhelper.New[Traces|Metrics|Logs]Exporter` with `exporterhelper.New[Traces|Metrics|Logs]ExporterWithContext` definition (#5914) +- Replace `processorhelper.New[Traces|Metrics|Logs]Exporter` + with `processorhelper.New[Traces|Metrics|Logs]ProcessorWithCreateSettings` definition (#5915) +- Replace `exporterhelper.New[Traces|Metrics|Logs]Exporter` + with `exporterhelper.New[Traces|Metrics|Logs]ExporterWithContext` definition (#5914) - Replace ``component.NewExtensionFactory`` with `component.NewExtensionFactoryWithStabilityLevel` definition (#5917) -- Set TLS 1.2 as default for `min_version` for TLS configuration in case this property is not defined (affects servers). (#5956) +- Set TLS 1.2 as default for `min_version` for TLS configuration in case this property is not defined (affects servers) + . (#5956) ### 🚩 Deprecations 🚩 -- Deprecate `processorhelper.New[Traces|Metrics|Logs]ProcessorWithCreateSettings` in favor of `processorhelper.New[Traces|Metrics|Logs]Exporter` (#5915) -- Deprecates `LogRecord.Flags()` and `LogRecord.SetFlags()` in favor of `LogRecord.FlagsStruct()` and `LogRecord.SetFlagsStruct()`. (#5972) -- Deprecate `exporterhelper.New[Traces|Metrics|Logs]ExporterWithContext` in favor of `exporterhelper.New[Traces|Metrics|Logs]Exporter` (#5914) +- Deprecate `processorhelper.New[Traces|Metrics|Logs]ProcessorWithCreateSettings` in favor + of `processorhelper.New[Traces|Metrics|Logs]Exporter` (#5915) +- Deprecates `LogRecord.Flags()` and `LogRecord.SetFlags()` in favor of `LogRecord.FlagsStruct()` + and `LogRecord.SetFlagsStruct()`. (#5972) +- Deprecate `exporterhelper.New[Traces|Metrics|Logs]ExporterWithContext` in favor + of `exporterhelper.New[Traces|Metrics|Logs]Exporter` (#5914) - Deprecate `component.NewExtensionFactoryWithStabilityLevel` in favor of `component.NewExtensionFactory` (#5917) - Deprecate `plog.SeverityNumber[UPPERCASE]` constants (#5927) - Deprecate `pcommon.Map.InsertNull` method (#5955) @@ -154,9 +167,10 @@ - `NewMetricDataPointFlagsStruct` -> `NewMetricDataPointFlags` - Deprecate builder distribution flags, use configuration. (#5946) - Enforce naming conventions for Invalid[Trace|Span]ID: (#5969) - - Deprecate funcs `pcommon.InvalidTraceID` and `pcommon.InvalidSpanID` in favor of vars `pcommon.EmptyTraceID` and `pcommon.EmptySpanID` + - Deprecate funcs `pcommon.InvalidTraceID` and `pcommon.InvalidSpanID` in favor of vars `pcommon.EmptyTraceID` + and `pcommon.EmptySpanID` - Deprecate `Update` and `Upsert` methods of `pcommon.Map` (#5975) -- Deprecated the current MetricDataPointFlags API. The new API provides functions to check and set Flags. (#5999) +- Deprecated the current MetricDataPointFlags API. The new API provides functions to check and set Flags. (#5999) - `NumberDataPoint.Flags` -> `NumberDataPoint.FlagsImmutable` - `HistogramDataPoint.Flags` -> `HistogramDataPoint.FlagsImmutable` - `ExponentialHistogramDataPoint.Flags` -> `ExponentialHistogramDataPoint.FlagsImmutable` @@ -166,7 +180,8 @@ ### 💡 Enhancements 💡 -- Added `MarshalerSizer` interface to `ptrace`, `plog`, and `pmetric` packages. `NewProtoMarshaler` now returns a `MarshalerSizer` (#5929) +- Added `MarshalerSizer` interface to `ptrace`, `plog`, and `pmetric` packages. `NewProtoMarshaler` now returns + a `MarshalerSizer` (#5929) - Add support to unmarshalls bytes into pmetric.Metrics with `jsoniter` in jsonUnmarshaler(#5433) - Add httpprovider to allow loading config files stored in HTTP (#5810) - Added `service.telemetry.traces.propagators` configuration to set propagators for collector's internal spans. (#5572) @@ -195,7 +210,8 @@ ### 🛑 Breaking changes 🛑 - Remove the InstrumentationLibrary to Scope translation (part of transition to OTLP 0.19). (#5819) - - This has a side effect that when sending JSON encoded telemetry using OTLP proto <= 0.15.0, telemetry will be dropped. + - This has a side effect that when sending JSON encoded telemetry using OTLP proto <= 0.15.0, telemetry will be + dropped. - Require the storage to be explicitly set for the (experimental) persistent queue (#5784) - Remove deprecated `confighttp.HTTPClientSettings.ToClientWithHost` (#5803) - Remove deprecated component stability helpers (#5802): @@ -227,12 +243,15 @@ - `component.ReceiverFactory.TracesReceiverStability` - `component.ReceiverFactory.MetricsReceiverStability` - `component.ReceiverFactory.LogsReceiverStability` -- Deprecate `obsreport.ProcessorSettings.Level` and `obsreport.ExporterSettings.Level`, use MetricsLevel from CreateSettings (#5824) -- Deprecate `processorhelper.New[Traces|Metrics|Logs]Processor` in favor of `processorhelper.New[Traces|Metrics|Logs]ProcessorWithCreateSettings` (#5833) +- Deprecate `obsreport.ProcessorSettings.Level` and `obsreport.ExporterSettings.Level`, use MetricsLevel from + CreateSettings (#5824) +- Deprecate `processorhelper.New[Traces|Metrics|Logs]Processor` in favor + of `processorhelper.New[Traces|Metrics|Logs]ProcessorWithCreateSettings` (#5833) - Deprecate MetricDataPointFlags.String(), no other pdata flags have this method (#5868) - Deprecates `FlagsStruct` in favor of `Flags` (#5842) - `FlagsStruct` -> `Flags` -- Deprecate `exporterhelper.New[Traces|Metrics|Logs]Exporter` in favor of `exporterhelper.New[Traces|Metrics|Logs]ExporterWithContext` (#5834) +- Deprecate `exporterhelper.New[Traces|Metrics|Logs]Exporter` in favor + of `exporterhelper.New[Traces|Metrics|Logs]ExporterWithContext` (#5834) ### 💡 Enhancements 💡 @@ -273,7 +292,7 @@ There isn't a valid core binary for this release. Use v0.57.2 instead. ### 🚩 Deprecations 🚩 -- Deprecated the current Flag API. The new API provides functions to check and set Flags (#5790) (#5602): +- Deprecated the current Flag API. The new API provides functions to check and set Flags (#5790) (#5602): - `NumberDataPoint.Flags` -> `NumberDataPoint.FlagsStruct` - `NumberDataPoint.SetFlags` -> `NumberDataPoint.FlagsStruct` - `HistogramDataPoint.Flags` -> `HistogramDataPoint.FlagsStruct` @@ -336,7 +355,8 @@ There isn't a valid core binary for this release. Use v0.57.2 instead. ### 🚩 Deprecations 🚩 -- Deprecate `service.ConfigServiceTelemetry`, `service.ConfigServiceTelemetryLogs`, and `service.ConfigServiceTelemetryMetrics` (#5565) +- Deprecate `service.ConfigServiceTelemetry`, `service.ConfigServiceTelemetryLogs`, + and `service.ConfigServiceTelemetryMetrics` (#5565) - Deprecate the following component functions to ensure a stability level is set (#5580): - `component.WithTracesExporter` -> `component.WithTracesExporterAndStabilityLevel` - `component.WithMetricsExporter` -> `component.WithMetricsExporterAndStabilityLevel` @@ -351,8 +371,8 @@ There isn't a valid core binary for this release. Use v0.57.2 instead. ### 💡 Enhancements 💡 -- Components stability levels are now logged. By default components which haven't defined their stability levels, or which are - unmaintained, deprecated or in development will log a message. (#5580) +- Components stability levels are now logged. By default components which haven't defined their stability levels, or + which are unmaintained, deprecated or in development will log a message. (#5580) - `exporter/logging`: Skip "bad file descriptor" sync errors (#5585) ### 🧰 Bug fixes 🧰 @@ -371,7 +391,8 @@ There isn't a valid core binary for this release. Use v0.57.2 instead. - Remove deprecated `config.MapConverter` (#5505) - Remove deprecated `config.Received` (#5505) - Remove deprecated `config.CloseFunc` (#5505) -- Deprecated `pcommon.Value.NewValueBytes` is brought back taking `pcommon.ImmutableByteSlice` as an argument instead of `[]byte` (#5299) +- Deprecated `pcommon.Value.NewValueBytes` is brought back taking `pcommon.ImmutableByteSlice` as an argument instead + of `[]byte` (#5299) ### 🚩 Deprecations 🚩 @@ -403,8 +424,10 @@ There isn't a valid core binary for this release. Use v0.57.2 instead. - Update sum field of exponential histograms to make it optional (#5530) - Remove redundant extension shutdown call (#5532) - Refactor pipelines builder, fix some issues (#5512) - - Unconfigured receivers are not identified, this was not a real problem in final binaries since the validation of the config catch this. - - Allow configurations to contain "unused" receivers. Receivers that are configured but not used in any pipeline, this was possible already for exporters and processors. + - Unconfigured receivers are not identified, this was not a real problem in final binaries since the validation of the + config catch this. + - Allow configurations to contain "unused" receivers. Receivers that are configured but not used in any pipeline, this + was possible already for exporters and processors. - Remove the enforcement/check that Receiver factories create the same instance for the same config. ## v0.53.0 Beta @@ -472,10 +495,9 @@ There isn't a valid core binary for this release. Use v0.57.2 instead. - Update OTLP to v0.17.0 (#5335) - Add optional min/max fields to histograms (#5399) - User-defined Resource attributes can be specified under `service.telemetry.resource` - configuration key and will be included as metric lables for own telemetry. - If `service.instance.id` is not specified it will be auto-generated. Previously - `service.instance.id` was always auto-generated, so the default of the new - behavior matches the old behavior. (#5402) + configuration key and will be included as metric lables for own telemetry. If `service.instance.id` is not specified + it will be auto-generated. Previously + `service.instance.id` was always auto-generated, so the default of the new behavior matches the old behavior. (#5402) ## v0.51.0 Beta @@ -535,13 +557,16 @@ There isn't a valid core binary for this release. Use v0.57.2 instead. - Deprecate global `featuregate` funcs in favor of `GetRegistry` and a public `Registry` type (#5160) ### 💡 Enhancements 💡 + - Add `jsoniter` Unmarshaller (#4817) - Extend config.Map.Unmarshal hook to check map key string to any TextUnmarshaler not only ComponentID (#5244) - Collector will no longer print error with stack trace when the collector is shutdown due to a context cancel. (#5258) ### 🧰 Bug fixes 🧰 -- Fix translation from otlp.Request to pdata representation, changes to the returned pdata not all reflected to the otlp.Request (#5197) + +- Fix translation from otlp.Request to pdata representation, changes to the returned pdata not all reflected to the + otlp.Request (#5197) - `exporterhelper` now properly consumes any remaining items on stop (#5203) - `pdata`: Fix copying of `Value` with `ValueTypeBytes` type (#5267) - `pdata`: Fix copying of metric fields of primitive items slice type (#5271) @@ -561,12 +586,13 @@ There isn't a valid core binary for this release. Use v0.57.2 instead. ### 🚩 Deprecations 🚩 -- All pdata related APIs from model (model/pdata, model/otlp and model/otlpgrpc) are deprecated in - favor of packages in the new pdata module separated by telemetry signal type (#5168) +- All pdata related APIs from model (model/pdata, model/otlp and model/otlpgrpc) are deprecated in favor of packages in + the new pdata module separated by telemetry signal type (#5168) - `model/pdata`, `model/otlp` -> `pdata/pcommon`, `pdata/plog`, `pdata/pmetric`, `pdata/ptrace` - `model/otlpgrpc` -> `pdata/plog/plogotlp`, `pdata/pmetric/pmetricotlp`, `pdata/ptrace/ptraceotlp` - Deprecate configmapprovider package, replace with mapconverter (#5167) -- Deprecate `service.MustNewConfigProvider` and `service.MustNewDefaultConfigProvider`in favor of `service.NewConfigProvider` (#4936) +- Deprecate `service.MustNewConfigProvider` and `service.MustNewDefaultConfigProvider`in favor + of `service.NewConfigProvider` (#4936) ### 💡 Enhancements 💡 @@ -588,9 +614,8 @@ There isn't a valid core binary for this release. Use v0.57.2 instead. - Remove pdata `InternalRep` deprecated funcs (#5018) - Remove service/defaultcomponents deprecated package (#5019) - Remove deprecated UseOpenTelemetryForInternalMetrics (#5026) -- Change outcome of `pdata.Value.MapVal()` and `pdata.Value.SliceVal()` functions misuse. In case of - type mismatch, they now return an invalid zero-initialized instance instead of a detached - collection (#5034) +- Change outcome of `pdata.Value.MapVal()` and `pdata.Value.SliceVal()` functions misuse. In case of type mismatch, they + now return an invalid zero-initialized instance instead of a detached collection (#5034) - OTLP JSON field changes following upgrade to OTLP v0.15.0: - "instrumentationLibraryLogs" is now "scopeLogs" - "instrumentationLibraryMetrics" is now "scopeMetrics" @@ -624,27 +649,21 @@ There isn't a valid core binary for this release. Use v0.57.2 instead. - Deprecate [Traces|Metrics|Logs][Reques|Response].Marshal in favor of `MarshalProto`. - Deprecate UnmarshalJSON[Traces|Metrics|Logs][Reques|Response] in favor of `UnmarshalProto`. - Deprecating following pdata methods/types following OTLP v0.15.0 upgrade (#5076): - - InstrumentationLibrary is now InstrumentationScope - - NewInstrumentationLibrary is now NewInstrumentationScope - - InstrumentationLibraryLogsSlice is now ScopeLogsSlice - - NewInstrumentationLibraryLogsSlice is now NewScopeLogsSlice - - InstrumentationLibraryLogs is now ScopeLogs - - NewInstrumentationLibraryLogs is now NewScopeLogs - - InstrumentationLibraryMetricsSlice is now ScopeMetricsSlice - - NewInstrumentationLibraryMetricsSlice is now NewScopeMetricsSlice - - InstrumentationLibraryMetrics is now ScopeMetrics - - NewInstrumentationLibraryMetrics is now NewScopeMetrics - - InstrumentationLibrarySpansSlice is now ScopeSpansSlice - - NewInstrumentationLibrarySpansSlice is now NewScopeSpansSlice - - InstrumentationLibrarySpans is now ScopeSpans - - NewInstrumentationLibrarySpans is now NewScopeSpans + - InstrumentationLibrary is now InstrumentationScope - NewInstrumentationLibrary is now NewInstrumentationScope - + InstrumentationLibraryLogsSlice is now ScopeLogsSlice - NewInstrumentationLibraryLogsSlice is now NewScopeLogsSlice + - InstrumentationLibraryLogs is now ScopeLogs - NewInstrumentationLibraryLogs is now NewScopeLogs - + InstrumentationLibraryMetricsSlice is now ScopeMetricsSlice - NewInstrumentationLibraryMetricsSlice is now + NewScopeMetricsSlice - InstrumentationLibraryMetrics is now ScopeMetrics - NewInstrumentationLibraryMetrics is now + NewScopeMetrics - InstrumentationLibrarySpansSlice is now ScopeSpansSlice - NewInstrumentationLibrarySpansSlice is + now NewScopeSpansSlice - InstrumentationLibrarySpans is now ScopeSpans - NewInstrumentationLibrarySpans is now + NewScopeSpans ### 💡 Enhancements 💡 - Add semconv definitions for v1.9.0 (#5090) -- Change outcome of `pdata.Metric.()` functions misuse. - In case of type mismatch, they don't panic right away but return an invalid zero-initialized - instance for consistency with other OneOf field accessors (#5035) +- Change outcome of `pdata.Metric.()` functions misuse. In case of type + mismatch, they don't panic right away but return an invalid zero-initialized instance for consistency with other OneOf + field accessors (#5035) - Update OTLP to v0.15.0 (#5064) - Adding support for transition from older versions of OTLP to OTLP v0.15.0 (#5085) @@ -693,8 +712,10 @@ There isn't a valid core binary for this release. Use v0.57.2 instead. - Deprecated funcs `config.DefaultConfig`, `confighttp.DefaultHTTPSettings`, `exporterhelper.DefaultTimeoutSettings`, `exporthelper.DefaultQueueSettings`, `exporterhelper.DefaultRetrySettings`, `testcomponents.DefaultFactories`, and - `scraperhelper.DefaultScraperControllerSettings` in favour for their `NewDefault` method to adhere to contribution guidelines (#4865) -- Deprecated funcs `componenthelper.StartFunc`, `componenthelper.ShutdownFunc` in favour of `component.StartFunc` and `component.ShutdownFunc` (#4803) + `scraperhelper.DefaultScraperControllerSettings` in favour for their `NewDefault` method to adhere to contribution + guidelines (#4865) +- Deprecated funcs `componenthelper.StartFunc`, `componenthelper.ShutdownFunc` in favour of `component.StartFunc` + and `component.ShutdownFunc` (#4803) - Move helpers from extensionhelper to component (#4805) - Deprecated `extensionhelper.CreateDefaultConfig` in favour of `component.ExtensionDefaultConfigFunc` - Deprecated `extensionhelper.CreateServiceExtension` in favour of `component.CreateExtensionFunc` @@ -726,11 +747,10 @@ There isn't a valid core binary for this release. Use v0.57.2 instead. ## 🧰 Bug fixes 🧰 - Initialized logger with collector to avoid potential race condition panic on `Shutdown` (#4827) -- In addition to traces, now logs and metrics processors will start the memory limiter. - Added thread-safe logic so only the first processor can launch the `checkMemLimits` go-routine and the last processor - that calls shutdown to terminate it; this is done per memory limiter instance. - Added memory limiter factory to cache initiated object and be reused by similar config. This guarantees a single - running `checkMemLimits` per config (#4886) +- In addition to traces, now logs and metrics processors will start the memory limiter. Added thread-safe logic so only + the first processor can launch the `checkMemLimits` go-routine and the last processor that calls shutdown to terminate + it; this is done per memory limiter instance. Added memory limiter factory to cache initiated object and be reused by + similar config. This guarantees a single running `checkMemLimits` per config (#4886) - Resolved race condition in collector when calling `Shutdown` (#4878) ## v0.45.0 Beta @@ -761,10 +781,9 @@ There isn't a valid core binary for this release. Use v0.57.2 instead. ### 🛑 Breaking changes 🛑 -- Updated to OTLP 0.12.0. Deprecated traces and metrics messages that existed - in 0.11.0 are no longer converted to the messages and fields that replaced the deprecated ones. - Received deprecated messages and fields will be now ignored. In OTLP/JSON in the - instrumentationLibraryLogs object the "logs" field is now named "logRecords" (#4724) +- Updated to OTLP 0.12.0. Deprecated traces and metrics messages that existed in 0.11.0 are no longer converted to the + messages and fields that replaced the deprecated ones. Received deprecated messages and fields will be now ignored. In + OTLP/JSON in the instrumentationLibraryLogs object the "logs" field is now named "logRecords" (#4724) - Deprecate `service.NewWindowsService`, add `service.NewSvcHandler` (#4783). ### 🚩 Deprecations 🚩 @@ -792,7 +811,8 @@ There isn't a valid core binary for this release. Use v0.57.2 instead. - Remove deprecated `configtest.LoadConfig` and `configtest.LoadConfigAndValidate` (#4659) - Move service.ConfigMapConverterFunc to config.MapConverterFunc (#4673) - Add context to config.MapConverterFunc (#4678) -- Builder: the skip compilation should only be supplied as a CLI flag. Previously, it was possible to specify that in the YAML file, contrary to the original intention (#4645) +- Builder: the skip compilation should only be supplied as a CLI flag. Previously, it was possible to specify that in + the YAML file, contrary to the original intention (#4645) - Builder: Remove deprecated config option module::core (#4693) - Remove deprecate flags --metrics-level and --metrics-addr (#4695) - Usages of `--metrics-level={VALUE}` can be replaced by `--set=service.telemetry.metrics.level={VALUE}`; @@ -815,7 +835,8 @@ There isn't a valid core binary for this release. Use v0.57.2 instead. - Expose experimental API `configmapprovider.NewExpandConverter()` (#4672) - `service.NewConfigProvider`: copy slice argument, disallow changes from caller to the input slice (#4729) -- `confighttp` and `configgrpc`: New config option `include_metadata` to persist request metadata/headers in `client.Info.Metadata` (experimental) (#4547) +- `confighttp` and `configgrpc`: New config option `include_metadata` to persist request metadata/headers + in `client.Info.Metadata` (experimental) (#4547) - Remove expand cases that cannot happen with config.Map (#4649) - Add `max_request_body_size` to confighttp.HTTPServerSettings (#4677) - Move `compression.go` into `confighttp.go` to internalize functions in `compression.go` file. (#4651) @@ -842,11 +863,11 @@ There isn't a valid core binary for this release. Use v0.57.2 instead. - `confighttp`: add client-side compression support. (#4441) - Each exporter should remove `compression` field if they have and should use `confighttp.HTTPClientSettings` -- Allow more zap logger configs: `disable_caller`, `disable_stacktrace`, `output_paths`, `error_output_paths`, `initial_fields` (#1048) +- Allow more zap logger configs: `disable_caller`, `disable_stacktrace`, `output_paths`, `error_output_paths` + , `initial_fields` (#1048) - Allow the custom zap logger encoding (#4532) - Collector self-metrics may now be configured through the configuration file. (#4069) - - CLI flags for configuring self-metrics are deprecated and will be removed - in a future release. + - CLI flags for configuring self-metrics are deprecated and will be removed in a future release. - `service.telemetry.metrics.level` and `service.telemetry.metrics.address` should be used to configure collector self-metrics. - `configauth`: add helpers to create new server authenticators. (#4558) @@ -870,7 +891,8 @@ There isn't a valid core binary for this release. Use v0.57.2 instead. - Remove `config.NewConfigMapFrom[File|Buffer]`, add testonly version (#4502) - `configtls`: TLS 1.2 is the new default mininum version (#4503) - `confighttp`: `ToServer` now accepts a `component.Host`, in line with gRPC's counterpart (#4514) -- CORS configuration for OTLP/HTTP receivers has been moved into a `cors:` block, instead of individual `cors_allowed_origins` and `cors_allowed_headers` settings (#4492) +- CORS configuration for OTLP/HTTP receivers has been moved into a `cors:` block, instead of + individual `cors_allowed_origins` and `cors_allowed_headers` settings (#4492) ### 💡 Enhancements 💡 @@ -889,7 +911,8 @@ There isn't a valid core binary for this release. Use v0.57.2 instead. ### 🛑 Breaking changes 🛑 -- Package `client` refactored (#4416) and auth data included in it (#4422). Final PR to be merged in the next release (#4423) +- Package `client` refactored (#4416) and auth data included in it (#4422). Final PR to be merged in the next release ( + # 4423) - Remove `pdata.AttributeMap.InitFromMap` (#4429) - Updated configgrpc `ToDialOptions` to support passing providers to instrumentation library (#4451) - Make state information propagation non-blocking on the collector (#4460) @@ -898,7 +921,8 @@ There isn't a valid core binary for this release. Use v0.57.2 instead. - Add semconv 1.7.0 and 1.8.0 (#4452) - Added `feature-gates` CLI flag for controlling feature gate state. (#4368) -- Add a default user-agent header to the OTLP/gRPC and OTLP/HTTP exporters containing collector build information (#3970) +- Add a default user-agent header to the OTLP/gRPC and OTLP/HTTP exporters containing collector build information ( + # 3970) ## v0.39.0 Beta @@ -924,10 +948,8 @@ There isn't a valid core binary for this release. Use v0.57.2 instead. - otlpexporter: Do not retry on PermissionDenied and Unauthenticated (#4349) - Enable configuring collector metrics through service config file. (#4069) - New `service::telemetry::metrics` structure added to configuration - - Existing metrics configuration CLI flags are deprecated and to be - removed in the future. - - `--metrics-prefix` is no longer operative; the prefix is determined by - the value of `service.buildInfo.Command`. + - Existing metrics configuration CLI flags are deprecated and to be removed in the future. + - `--metrics-prefix` is no longer operative; the prefix is determined by the value of `service.buildInfo.Command`. - `--add-instance-id` is no longer operative; an instance ID will always be added. - Remove deprecated funcs `consumererror.As[Traces|Metrics|Logs]` (#4364) - Remove support to expand env variables in default configs (#4366) @@ -945,7 +967,8 @@ There isn't a valid core binary for this release. Use v0.57.2 instead. ### 🛑 Breaking changes 🛑 -- Removed `configauth.HTTPClientAuthenticator` and `configauth.GRPCClientAuthenticator` in favor of `configauth.ClientAuthenticator`. (#4255) +- Removed `configauth.HTTPClientAuthenticator` and `configauth.GRPCClientAuthenticator` in favor + of `configauth.ClientAuthenticator`. (#4255) - Rename `parserprovider.MapProvider` as `config.MapProvider`. (#4178) - Rename `parserprovider.Watchable` as `config.WatchableMapProvider`. (#4178) - Remove deprecated no-op flags to setup Collector's logging "--log-level", "--log-profile", "--log-format". (#4213) @@ -974,10 +997,12 @@ There isn't a valid core binary for this release. Use v0.57.2 instead. - Rename `configparser.Parser` as `config.Map` (#4075) - Rename `component.DefaultBuildInfo()` to `component.NewDefaultBuildInfo()` (#4129) - Rename `consumererror.Permanent` to `consumererror.NewPermanent` (#4118) -- Rename `config.NewID` to `config.NewComponentID` and `config.NewIDFromString` to `config.NewComponentIDFromString` (#4137) +- Rename `config.NewID` to `config.NewComponentID` and `config.NewIDFromString` to `config.NewComponentIDFromString` ( + # 4137) - Rename `config.NewIDWithName` to `config.NewComponentIDWithName` (#4151) - Move `extension/storage` to `extension/experimental/storage` (#4082) -- Rename `obsreporttest.SetupRecordedMetricsTest()` to `obsreporttest.SetupTelemetry()` and `obsreporttest.TestTelemetrySettings` to `obsreporttest.TestTelemetry` (#4157) +- Rename `obsreporttest.SetupRecordedMetricsTest()` to `obsreporttest.SetupTelemetry()` + and `obsreporttest.TestTelemetrySettings` to `obsreporttest.TestTelemetry` (#4157) ### 💡 Enhancements 💡 @@ -1035,7 +1060,8 @@ There isn't a valid core binary for this release. Use v0.57.2 instead. ### 🛑 Breaking changes 🛑 -- Artifacts are no longer published in this repository, check [here](https://github.com/open-telemetry/opentelemetry-collector-releases) (#3941) +- Artifacts are no longer published in this repository, + check [here](https://github.com/open-telemetry/opentelemetry-collector-releases) (#3941) - Remove deprecated `tracetranslator.AttributeValueToString` and `tracetranslator.AttributeMapToMap` (#3873) - Change semantic conventions for status (code, msg) as per specifications (#3872) - Move `fileexporter` to contrib (#3474) @@ -1301,7 +1327,8 @@ This release is marked as "bad" since the metrics pipelines will produce bad dat ### 🛑 Breaking changes 🛑 -- Change `Marshal` signatures in kafkaexporter's Marshalers to directly convert pdata to `sarama.ProducerMessage` (#3162) +- Change `Marshal` signatures in kafkaexporter's Marshalers to directly convert pdata to `sarama.ProducerMessage` ( + # 3162) - Remove `tracetranslator.DetermineValueType`, only used internally by Zipkin (#3114) - Remove OpenCensus conventions, should not be used (#3113) - Remove Zipkin specific translation constants, move to internal (#3112) @@ -1347,7 +1374,8 @@ This release is marked as "bad" since the metrics pipelines will produce bad dat - Change `With*Unmarshallers` signatures in Kafka exporter/receiver (#2973) - Rename `marshall` to `marshal` in all the occurrences (#2977) -- Remove `componenterror.ErrAlreadyStarted` and `componenterror.ErrAlreadyStopped`, components should not protect against this, Service will start/stop once. +- Remove `componenterror.ErrAlreadyStarted` and `componenterror.ErrAlreadyStopped`, components should not protect + against this, Service will start/stop once. - Rename `ApplicationStartInfo` to `BuildInfo` - Rename `ApplicationStartInfo.ExeName` to `BuildInfo.Command` - Rename `ApplicationStartInfo.LongName` to `BuildInfo.Description` @@ -1361,7 +1389,8 @@ This release is marked as "bad" since the metrics pipelines will produce bad dat - `kafka` exporter: Add logs support (#2943) - Update mdatagen to create factories of init instead of new (#2978) - `zipkin` receiver: Reduce the judgment of zipkin v1 version (#2990) -- Custom authenticator logic to accept a `component.Host` which will extract the authenticator to use based on a new authenticator name property (#2767) +- Custom authenticator logic to accept a `component.Host` which will extract the authenticator to use based on a new + authenticator name property (#2767) - `prometheusremotewrite` exporter: Add `resource_to_telemetry_conversion` config option (#3031) - `logging` exporter: Extract OTLP text logging (#3082) - Format timestamps as strings instead of int in otlptext output (#3088) @@ -1387,7 +1416,9 @@ This release is marked as "bad" since the metrics pipelines will produce bad dat - Rename ForEach (in pdata) with Range to be consistent with sync.Map (#2931) - Rename `componenthelper.Start` to `componenthelper.StartFunc` (#2880) - Rename `componenthelper.Stop` to `componenthelper.StopFunc` (#2880) -- Remove `exporterheleper.WithCustomUnmarshaler`, `processorheleper.WithCustomUnmarshaler`, `receiverheleper.WithCustomUnmarshaler`, `extensionheleper.WithCustomUnmarshaler`, implement `config.CustomUnmarshaler` interface instead (#2867) +- Remove `exporterheleper.WithCustomUnmarshaler`, `processorheleper.WithCustomUnmarshaler` + , `receiverheleper.WithCustomUnmarshaler`, `extensionheleper.WithCustomUnmarshaler`, + implement `config.CustomUnmarshaler` interface instead (#2867) - Remove `component.CustomUnmarshaler` implement `config.CustomUnmarshaler` interface instead (#2867) - Remove `testutil.HostPortFromAddr`, users can write their own parsing helper (#2919) - Remove `configparser.DecodeTypeAndName`, use `config.IDFromString` (#2869) @@ -1490,7 +1521,8 @@ This release is marked as "bad" since the metrics pipelines will produce bad dat - Rename pdata Size to OtlpProtoSize (#2726) - Rename [Traces|Metrics|Logs]Consumer to [Traces|Metrics|Logs] (#2761) - Remove public access for `componenttest.Example*` components: - - Users of these structs for testing configs should use the newly added `componenttest.Nop*` (update all components name in the config `example*` -> `nop` and use `componenttest.NopComponents()`). + - Users of these structs for testing configs should use the newly added `componenttest.Nop*` (update all components + name in the config `example*` -> `nop` and use `componenttest.NopComponents()`). - Users of these structs for sink like behavior should use `consumertest.*Sink`. ### 💡 Enhancements 💡 @@ -1509,7 +1541,8 @@ This release is marked as "bad" since the metrics pipelines will produce bad dat ### 🧰 Bug fixes 🧰 -- `prometheus` exporter: Fix to work with standard labels that follow the naming convention of using periods instead of underscores (#2707) +- `prometheus` exporter: Fix to work with standard labels that follow the naming convention of using periods instead of + underscores (#2707) - Propagate name and transport for `prometheus` receiver and exporter (#2680) - `zipkin` receiver: Ensure shutdown correctness (#2765) @@ -1528,7 +1561,8 @@ This release is marked as "bad" since the metrics pipelines will produce bad dat ### 💡 Enhancements 💡 -- `prometheus` exporter: Re-implement on top of `github.com/prometheus/client_golang/prometheus` and add `metric_expiration` option +- `prometheus` exporter: Re-implement on top of `github.com/prometheus/client_golang/prometheus` and + add `metric_expiration` option - `logging` exporter: Add support for AttributeMap (#2609) - Add semantic conventions for instrumentation library (#2602) @@ -1575,7 +1609,8 @@ This release is marked as "bad" since the metrics pipelines will produce bad dat ### 🛑 Breaking changes 🛑 - Remove deprecated `queued_retry` processor -- Remove deprecated configs from `resource` processor: `type` (set "opencensus.type" key in "attributes.upsert" map instead) and `labels` (use "attributes.upsert" instead). +- Remove deprecated configs from `resource` processor: `type` (set "opencensus.type" key in "attributes.upsert" map + instead) and `labels` (use "attributes.upsert" instead). ### 💡 Enhancements 💡 @@ -1630,8 +1665,8 @@ This release is marked as "bad" since the metrics pipelines will produce bad dat ### 💡 Enhancements 💡 -- Change default OTLP/gRPC port number to 4317, also continue receiving on legacy port - 55680 during transition period (#2104). +- Change default OTLP/gRPC port number to 4317, also continue receiving on legacy port 55680 during transition period ( + # 2104). - `kafka` exporter: Add support for exporting metrics as otlp Protobuf. (#1966) - Move scraper helpers to its own `scraperhelper` package (#2185) - Add `componenthelper` package to help build components (#2186) @@ -1802,7 +1837,8 @@ This release is marked as "bad" since the metrics pipelines will produce bad dat - Rename service.Start() to Run() since it's a blocking call - Fix slice Append to accept by value the element in pdata -- Change CreateTraceProcessor and CreateMetricsProcessor to use the same parameter order as receivers/logs processor and exporters. +- Change CreateTraceProcessor and CreateMetricsProcessor to use the same parameter order as receivers/logs processor and + exporters. - Prevent accidental use of LogsToOtlp and LogsFromOtlp and the OTLP data structs (#1703) - Remove SetType from configmodels, ensure all registered factories set the type in config (#1798) - Move process telemetry to service/internal (#1794) @@ -1859,7 +1895,8 @@ This release is marked as "bad" since the metrics pipelines will produce bad dat ### 🚀 New components 🚀 -- `prometheusremotewrite` exporter: Send metrics data in Prometheus TimeSeries format to Cortex or any Prometheus (#1544) +- `prometheusremotewrite` exporter: Send metrics data in Prometheus TimeSeries format to Cortex or any Prometheus ( + # 1544) - `kafka` receiver: Receive traces from Kafka (#1410) ### 💡 Enhancements 💡 @@ -1912,7 +1949,8 @@ This release is marked as "bad" since the metrics pipelines will produce bad dat ### 🚀 New components 🚀 - Receivers - - `fluentfoward` runs a TCP server that accepts events via the [Fluent Forward protocol](https://github.com/fluent/fluentd/wiki/Forward-Protocol-Specification-v1) (#1173) + - `fluentfoward` runs a TCP server that accepts events via + the [Fluent Forward protocol](https://github.com/fluent/fluentd/wiki/Forward-Protocol-Specification-v1) (#1173) - Exporters - `kafka` exports traces to Kafka (#1439) - Extensions @@ -1948,7 +1986,8 @@ This release is marked as "bad" since the metrics pipelines will produce bad dat ### 🛑 Breaking changes 🛑 -- Renamed the metrics generated by `hostmetrics` receiver to match the (currently still pending) OpenTelemetry system metric conventions (#1261) (#1269) +- Renamed the metrics generated by `hostmetrics` receiver to match the (currently still pending) OpenTelemetry system + metric conventions (#1261) (#1269) - Removed `vmmetrics` receiver (#1282) - Removed `cpu` scraper `report_per_cpu` config option (#1326) @@ -1962,7 +2001,8 @@ This release is marked as "bad" since the metrics pipelines will produce bad dat - Added log support for `memorylimiter` processor (#1291) and `logging` exporter (#1298) - Always add tags for `observability`, other metrics may use them (#1312) - Added metrics support (#1313) and allow partial retries in `queued_retry` processor (#1297) -- Update `resource` processor: introduce `attributes` config parameter to specify actions on attributes similar to `attributes` processor, old config interface is deprecated (#1315) +- Update `resource` processor: introduce `attributes` config parameter to specify actions on attributes similar + to `attributes` processor, old config interface is deprecated (#1315) - Update memory state labels for non-Linux OSs (#1325) - Ensure tcp connection value is provided for all states, even when count is 0 (#1329) - Set `batch` processor channel size to num cpus (#1330) @@ -2034,7 +2074,8 @@ Released 2020-06-16 ### 🚀 New components 🚀 - Receivers - - `hostmetrics` receiver with CPU (#862), disk (#921), load (#974), filesystem (#926), memory (#911), network (#930), and virtual memory (#989) support + - `hostmetrics` receiver with CPU (#862), disk (#921), load (#974), filesystem (#926), memory (#911), network (#930), + and virtual memory (#989) support - Processors - `batch` for batching received metrics (#1060) - `filter` for filtering (dropping) received metrics (#1001) @@ -2091,12 +2132,13 @@ Released 2020-03-30 ### Breaking changes -- Make prometheus receiver config loading strict. #697 - Prometheus receiver will now fail fast if the config contains unused keys in it. +- Make prometheus receiver config loading strict. #697 Prometheus receiver will now fail fast if the config contains + unused keys in it. ### Changes and fixes -- Enable best effort serve by default of Prometheus Exporter (https://github.com/orijtech/prometheus-go-metrics-exporter/pull/6) +- Enable best effort serve by default of Prometheus + Exporter (https://github.com/orijtech/prometheus-go-metrics-exporter/pull/6) - Fix null pointer exception in the logging exporter #743 - Remove unnecessary condition to have at least one processor #744 @@ -2243,14 +2285,25 @@ This is the first alpha release of OpenTelemetry Service. Docker image: omnition/opentelemetry-service:v0.0.1 [v0.3.0]: https://github.com/open-telemetry/opentelemetry-collector/compare/v0.2.10...v0.3.0 + [v0.2.10]: https://github.com/open-telemetry/opentelemetry-collector/compare/v0.2.8...v0.2.10 + [v0.2.8]: https://github.com/open-telemetry/opentelemetry-collector/compare/v0.2.7...v0.2.8 + [v0.2.7]: https://github.com/open-telemetry/opentelemetry-collector/compare/v0.2.6...v0.2.7 + [v0.2.6]: https://github.com/open-telemetry/opentelemetry-collector/compare/v0.2.5...v0.2.6 + [v0.2.5]: https://github.com/open-telemetry/opentelemetry-collector/compare/v0.2.4...v0.2.5 + [v0.2.4]: https://github.com/open-telemetry/opentelemetry-collector/compare/v0.2.3...v0.2.4 + [v0.2.3]: https://github.com/open-telemetry/opentelemetry-collector/compare/v0.2.2...v0.2.3 + [v0.2.2]: https://github.com/open-telemetry/opentelemetry-collector/compare/v0.2.0...v0.2.2 + [v0.2.0]: https://github.com/open-telemetry/opentelemetry-collector/compare/v0.0.2...v0.2.0 + [v0.0.2]: https://github.com/open-telemetry/opentelemetry-collector/compare/v0.0.1...v0.0.2 + [v0.0.1]: https://github.com/open-telemetry/opentelemetry-collector/tree/v0.0.1 diff --git a/pdata/pcommon/common.go b/pdata/pcommon/common.go index 15617076f30..8dc574c5daf 100644 --- a/pdata/pcommon/common.go +++ b/pdata/pcommon/common.go @@ -79,7 +79,7 @@ func (avt ValueType) String() string { // // func f1(val Value) { val.SetInt(234) } // func f2() { -// v := NewValueString("a string") +// v := NewValueStr("a string") // f1(v) // _ := v.Type() // this will return ValueTypeInt // } @@ -93,11 +93,14 @@ func NewValueEmpty() Value { return newValue(&otlpcommon.AnyValue{}) } -// NewValueString creates a new Value with the given string value. -func NewValueString(v string) Value { +// NewValueStr creates a new Value with the given string value. +func NewValueStr(v string) Value { return newValue(&otlpcommon.AnyValue{Value: &otlpcommon.AnyValue_StringValue{StringValue: v}}) } +// Deprecated: [0.62.0] Use NewValueStr instead. +var NewValueString = NewValueStr + // NewValueInt creates a new Value with the given int64 value. func NewValueInt(v int64) Value { return newValue(&otlpcommon.AnyValue{Value: &otlpcommon.AnyValue_IntValue{IntValue: v}}) diff --git a/pdata/pcommon/common_test.go b/pdata/pcommon/common_test.go index 61583bcfe7f..415f07df5b8 100644 --- a/pdata/pcommon/common_test.go +++ b/pdata/pcommon/common_test.go @@ -27,7 +27,7 @@ import ( ) func TestValue(t *testing.T) { - v := NewValueString("abc") + v := NewValueStr("abc") assert.EqualValues(t, ValueTypeStr, v.Type()) assert.EqualValues(t, "abc", v.Str()) @@ -89,7 +89,7 @@ func TestValueMap(t *testing.T) { assert.Equal(t, 1, m2.Len()) got, exists = m2.Get("key_in_child") assert.True(t, exists) - assert.Equal(t, NewValueString("somestr"), got) + assert.Equal(t, NewValueStr("somestr"), got) // Insert the second map as a child. This should perform a deep copy. assert.EqualValues(t, 2, m1.MapVal().Len()) @@ -105,26 +105,26 @@ func TestValueMap(t *testing.T) { assert.EqualValues(t, 1, m2.Len()) got, exists = m2.Get("key_in_child") assert.True(t, exists) - assert.Equal(t, NewValueString("somestr2"), got) + assert.Equal(t, NewValueStr("somestr2"), got) // The child map inside m1 should be modified. childMap, childMapExists := m1.MapVal().Get("child_map") require.True(t, childMapExists) got, exists = childMap.Map().Get("key_in_child") require.True(t, exists) - assert.Equal(t, NewValueString("somestr2"), got) + assert.Equal(t, NewValueStr("somestr2"), got) // Now modify the inserted map (not the source) childMap.MapVal().PutString("key_in_child", "somestr3") assert.EqualValues(t, 1, childMap.MapVal().Len()) got, exists = childMap.MapVal().Get("key_in_child") require.True(t, exists) - assert.Equal(t, NewValueString("somestr3"), got) + assert.Equal(t, NewValueStr("somestr3"), got) // The source child map should be modified. got, exists = m2.Get("key_in_child") require.True(t, exists) - assert.Equal(t, NewValueString("somestr3"), got) + assert.Equal(t, NewValueStr("somestr3"), got) removed := m1.Map().Remove("double_key") assert.True(t, removed) @@ -180,14 +180,14 @@ func TestValueEqual(t *testing.T) { av2 := NewValueEmpty() assert.True(t, av1.Equal(av2)) - av2 = NewValueString("abc") + av2 = NewValueStr("abc") assert.False(t, av1.Equal(av2)) assert.False(t, av2.Equal(av1)) - av1 = NewValueString("abc") + av1 = NewValueStr("abc") assert.True(t, av1.Equal(av2)) - av2 = NewValueString("edf") + av2 = NewValueStr("edf") assert.False(t, av1.Equal(av2)) av2 = NewValueInt(123) @@ -755,7 +755,7 @@ func TestValueSlice(t *testing.T) { a2.Slice().AppendEmpty().SetStr("somestr") assert.EqualValues(t, 1, a2.Slice().Len()) - assert.EqualValues(t, NewValueString("somestr"), a2.Slice().At(0)) + assert.EqualValues(t, NewValueStr("somestr"), a2.Slice().At(0)) // Insert the second array as a child. a2.CopyTo(a1.Slice().AppendEmpty()) @@ -806,7 +806,7 @@ func TestAsString(t *testing.T) { }{ { name: "string", - input: NewValueString("string value"), + input: NewValueStr("string value"), expected: "string value", }, { @@ -881,7 +881,7 @@ func TestValueAsRaw(t *testing.T) { }{ { name: "string", - input: NewValueString("value"), + input: NewValueStr("value"), expected: "value", }, { @@ -961,7 +961,7 @@ func TestNewValueFromRaw(t *testing.T) { { name: "string", input: "text", - expected: NewValueString("text"), + expected: NewValueStr("text"), }, { name: "int", @@ -1079,7 +1079,7 @@ func TestNewValueFromRaw(t *testing.T) { name: "invalid value", input: ValueTypeDouble, expected: (func() Value { - return NewValueString("") + return NewValueStr("") })(), }, }