You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There's a lot of duplication among the tracing SDK and the logs SDK. One of the areas of duplication is with LogRecord#validate_attributes. This is very similar to the code in Internal.valid_attributes.
Once the logs SDK is stable and we're ready to bring it into the main SDK gem, refactor Internal.valid_attributes for use with both signals. Depending on the timing, this may also be helpful for metric attribute validation as well.
👋 This issue has been marked as stale because it has been open with no activity. You can: comment on the issue or remove the stale label to hold stale off for a while, add the keep label to hold stale off permanently, or do nothing. If you do nothing this issue will be closed eventually by the stale bot.
There's a lot of duplication among the tracing SDK and the logs SDK. One of the areas of duplication is with
LogRecord#validate_attributes
. This is very similar to the code inInternal.valid_attributes
.Once the logs SDK is stable and we're ready to bring it into the main SDK gem, refactor
Internal.valid_attributes
for use with both signals. Depending on the timing, this may also be helpful for metric attribute validation as well.This issue originated from this comment: https://github.com/open-telemetry/opentelemetry-ruby/pull/1696/files#r1759599051
The text was updated successfully, but these errors were encountered: