oscerd opened a new pull request, #24974:
URL: https://github.com/apache/camel/pull/24974

   ### Motivation
   
   [CAMEL-23460](https://issues.apache.org/jira/browse/CAMEL-23460): follow-up 
to the AWS span-decorator work (CAMEL-23387) — add equivalent `camel-telemetry` 
span decorators for the Google Cloud components. This is the **first of the 
issue's suggested batches** (messaging & storage).
   
   ### Changes
   
   Four new decorators following the AWS pattern:
   
   | Decorator | Base | Tags emitted |
   |---|---|---|
   | `GooglePubsubSpanDecorator` | `AbstractMessagingSpanDecorator` | messageId 
(destination via base), orderingKey, ackId, deliveryAttempt |
   | `GoogleStorageSpanDecorator` | `AbstractSpanDecorator` | operation, 
bucketName, objectName, destinationBucketName, destinationObjectName |
   | `GoogleBigQuerySpanDecorator` | `AbstractSpanDecorator` | tableId, 
tableSuffix, partitionDecorator, jobId |
   | `GoogleFirestoreSpanDecorator` | `AbstractSpanDecorator` | operation, 
collectionName, documentId |
   
   - Header constants are **mirrored** from each component's `*Constants` 
interface (with a Javadoc `@link` back to the source), so `camel-telemetry` 
gains no hard dependency on the Google component modules — identical to the AWS 
decorators.
   - **Tag minimization** per the rules established in the AWS work: only 
request targets and operational identifiers are tagged; anything that may carry 
secrets/PII/large payloads is deliberately dropped — PubSub message 
`attributes`, Firestore `queryValue`, Storage signed `downloadLink` / 
`kmsKeyName`, BigQuery `translatedQuery`, etc.
   - Registered alphabetically in 
`META-INF/services/org.apache.camel.telemetry.SpanDecorator`.
   - One unit test per decorator (header-to-tag extraction), mirroring 
`AwsCwSpanDecoratorTest`.
   
   **Scope note:** `camel-google-pubsub-lite` appears in the issue's component 
list but is **not a component module** in the tree (no sources, no pom, no 
catalog entry), so it is omitted. The remaining two batches from the issue — 
workspace/productivity (Calendar, Drive, Mail, Sheets) and AI/ML & misc 
(Speech-to-Text, Text-to-Speech, Vision, VertexAI, Functions, Secret Manager) — 
will follow as separate PRs.
   
   ### Testing
   
   - `mvn clean install` in `components/camel-telemetry`: all tests pass (4 new 
+ existing).
   - Full reactor `mvn clean install -DskipTests` from root: success, no stale 
generated files.
   
   Metadata/observability addition — main/4.22.0 only, no backport.
   
   _Claude Code on behalf of Andrea Cosentino (@oscerd)._
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to