Skip to content

fix(reader): Correctly handle totals row#415

Merged
tkaemming merged 3 commits into
masterfrom
reader-date-formatting-fix
Aug 14, 2019
Merged

fix(reader): Correctly handle totals row#415
tkaemming merged 3 commits into
masterfrom
reader-date-formatting-fix

Conversation

@tkaemming

Copy link
Copy Markdown
Contributor

Fixes a regression introduced with #414 when rewriting date types in the totals row.

Before:

(snuba) ted@veneno % git co master
Switched to branch 'master'
(snuba) ted@veneno % python -m mypy --follow-imports skip snuba/reader.py
snuba/reader.py:52: error: Unsupported target for indexed assignment
snuba/reader.py:52: error: Value of type "Collection[str]" is not indexable
snuba/reader.py:55: error: Unsupported target for indexed assignment
snuba/reader.py:56: error: Value of type "Collection[str]" is not indexable

After:

(snuba) ted@veneno % git co reader-date-formatting-fix
Switched to branch 'reader-date-formatting-fix'
(snuba) ted@veneno % python -m mypy --follow-imports skip snuba/reader.py

@tkaemming
tkaemming requested a review from a team August 14, 2019 19:16
@tkaemming
tkaemming merged commit 59e6991 into master Aug 14, 2019
@tkaemming
tkaemming deleted the reader-date-formatting-fix branch August 14, 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