-
Notifications
You must be signed in to change notification settings - Fork 2.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Create otelarrow exporter instance per distinct metadata combos #34178
Comments
Pinging code owners:
See Adding Labels via Comments if you do not have permissions to add labels yourself. |
@kristinapathak Thanks for filing! I agree with the stated goal here. While the Arrow components can pass headers on a per-request level, this can present authorization challenges for vendors that would prefer to authorize based on HTTP-level headers. As you wrote, this sounds like functionality that could be implemented in an exporterhelper module, but there may not be many components that want this. The only reservation I have is that this method will tie us to the legacy style of batching in processor components and will restrict us from using the new exporterhelper-based batching. Ideally, I think we could look for ways to improve exporterhelper to support both batching and exporting by metadata, but that can be a future project. I support adding this functiolnaity directly in the Arrow exporter especially if the logic can be kept mostly separate, which seems natural. |
This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping Pinging code owners:
See Adding Labels via Comments if you do not have permissions to add labels yourself. |
…etadataKeys (open-telemetry#34827) **Description:** This PR forks open-telemetry#34235 and adds unit tests. **Link to tracking Issue:** open-telemetry#34178 --------- Co-authored-by: kristina.pathak <kristina.m.pathak@gmail.com> Co-authored-by: Joshua MacDonald <jmacd@users.noreply.github.com>
Component(s)
exporter/otelarrow
Is your feature request related to a problem? Please describe.
I want to open Arrow streams where specified metadata sent per stream message all match on a per stream basis, similar to configuring
metadata_keys
in the batch processor.Describe the solution you'd like
Similar to the batch processor, add the below configuration fields:
Then, for each unique combination of metadata keys, an arrow exporter object is created. This object creates the number of streams specified by
num_streams
. The maximum possible number of open streams would bemetadata_cardinality_limit * num_streams
.Note: there is nothing arrow exporter specific about this logic. It could be used for any exporter where unique objects are desired per metadata combination.
Describe alternatives you've considered
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: