Skip to content

Commit

Permalink
Deprecate LogRecord.Name field
Browse files Browse the repository at this point in the history
The field is removed from the spec: open-telemetry/opentelemetry-specification#2271

I am deprecating it for now, to give some time to existing users to get rid
of it (most notably in the Collector).

This will be removed soon (after the Collector cleanup).
  • Loading branch information
tigrannajaryan committed Feb 3, 2022
1 parent f8c59fe commit 4cd8f79
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ Full list of differences found in [this compare](https://github.com/open-telemet

### Removed

* Remove if no changes for this section before release.
* Deprecate LogRecord.Name field. (#357)

## 0.12.0 - 2022-01-19

Expand Down
2 changes: 1 addition & 1 deletion opentelemetry/proto/logs/v1/logs.proto
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,7 @@ message LogRecord {
// Short event identifier that does not contain varying parts. Name describes
// what happened (e.g. "ProcessStarted"). Recommended to be no longer than 50
// characters. Not guaranteed to be unique in any way. [Optional].
string name = 4;
string name = 4 [deprecated = true];

// A value containing the body of the log record. Can be for example a human-readable
// string message (including multi-line) describing the event in a free form or it can
Expand Down

0 comments on commit 4cd8f79

Please sign in to comment.