Skip to content

Conversation

@jack-berg
Copy link
Member

Followup to #426.

Common themes:

  • For all properties in a SDK extension plugin, set defaultBehavior: ignore.
  • For properties with complex types, where those types have defaults of their own for each property, set defaultBeahvior: default values as described in <type> are used

There weren't too many default behaviors that were completely missing. Mostly just dots that needed to be connected.

There was only one case where the defaults were unspecified and not obvious - the jaeger remote sampler. I made endpoint and initial_sampler required, and set the defaultBehavior: 60000 is used for internal (the default in java but not defined in the spec). This case isn't too important because we marked jaeger remote sampler as experimental in #363.

@jack-berg
Copy link
Member Author

Closing in favor of #439.

@jack-berg jack-berg closed this Dec 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant