Skip to content

ref: Use explicit argument references in query splitter#412

Merged
tkaemming merged 1 commit into
masterfrom
split-use-parameters
Aug 13, 2019
Merged

ref: Use explicit argument references in query splitter#412
tkaemming merged 1 commit into
masterfrom
split-use-parameters

Conversation

@tkaemming

Copy link
Copy Markdown
Contributor

This is a follow up to #388 (comment).

@tkaemming
tkaemming requested review from a team and lynnagara August 13, 2019 22:01

@tkaemming tkaemming left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm (probably) about to make some changes that will change the signature of parse_and_run_query, which will then change this function signature as well, so I wanted to get this out of the way first. Even if that change to parse_and_run_query never comes to fruition, this shouldn't have been written this way in the first place.

@lynnagara lynnagara left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@tkaemming
tkaemming merged commit 4b36a88 into master Aug 13, 2019
@tkaemming
tkaemming deleted the split-use-parameters branch August 13, 2019 22:07
phacops pushed a commit that referenced this pull request Jun 10, 2026
## Bump sentry-conventions to 0.11.0

Updates `sentry-conventions` from 0.10.0 to
[0.11.0](https://github.com/getsentry/sentry-conventions/releases/tag/0.11.0).

### Changes in 0.11.0

**New Features**
- Add `session.id` attribute (#412)
- Update several attributes to latest OTel representation (#418)
- Add `db.operation.batch.size` (#407)
- Add `app.vitals.start.prewarmed` attribute (#379)
- Add and deprecate runtime context attributes (#383)
- Add Android Runtime (ART) GC and memory attributes (#382)
- Add `db.stored_procedure.name` and ensure span name attribute
consistency (#398)
- Add Cloudflare SDK attributes (#392)
- Add missing GCP and FaaS attributes (#403)

**Bug Fixes**
- Set `pii: 'maybe'` on `faas` string attributes (#415)
- Add missing Cloudflare visibility attributes (#408)

**Internal**
- Remove `sdks` field from attribute schema and definitions (#410)
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.

2 participants