Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Trask Stalnaker <[email protected]>
  • Loading branch information
lmolkova and trask authored Oct 17, 2024
1 parent 9fb01e2 commit f604270
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions model/database/registry.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -49,14 +49,14 @@ groups:
stability: experimental # RELEASE CANDIDATE
brief: >
The name of the operation or command being executed.
note: >
note: |
It is RECOMMENDED to capture the value as provided by the application
without attempting to do any case normalization.
A single database query may involve multiple operations. If the operation
name is parsed from the query text, it SHOULD only be captured for queries that
contain a single operation or when the operation name describing the
whole query is available by other means (such as SQL comments).
whole query is available by other means.
For batch operations, if the individual operations are known to have the same operation name
then that operation name SHOULD be used prepended by `BATCH `,
Expand Down Expand Up @@ -111,8 +111,8 @@ groups:
a class of database queries and is useful as a grouping key,
especially when analyzing telemetry for database calls involving complex queries.
Summary may be available to the instrumentation through SQL comment,
instrumentation hooks, or other means. If it is not available, instrumentations
Summary may be available to the instrumentation through
instrumentation hooks or other means. If it is not available, instrumentations
that support query parsing SHOULD generate a summary following
[Generating query summary](../../docs/database/database-spans.md#generating-a-summary-of-the-quey-text)
section.
Expand Down

0 comments on commit f604270

Please sign in to comment.