Skip to content
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

deps: update cockroachdb/errors #77525

Merged
merged 1 commit into from
Mar 11, 2022
Merged

Conversation

knz
Copy link
Contributor

@knz knz commented Mar 9, 2022

Fixes #76571

This upgrades to v1.9.0 and enhances Sentry reports as requested by
the SQL queries team.

Release justification: low risk, high benefit changes to existing functionality

Release note: None

@knz knz requested a review from yuzefovich March 9, 2022 10:45
@blathers-crl blathers-crl bot added the T-server-and-security DB Server & Security label Mar 9, 2022
@cockroach-teamcity
Copy link
Member

This change is Reviewable

Copy link
Member

@yuzefovich yuzefovich left a comment

Choose a reason for hiding this comment

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

Thanks! :lgtm:

Reviewed 4 of 4 files at r1, all commit messages.
Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained (waiting on @knz)

@knz
Copy link
Contributor Author

knz commented Mar 11, 2022

TFYR!

bors r=yuzefovich

@craig
Copy link
Contributor

craig bot commented Mar 11, 2022

Build failed:

@knz knz force-pushed the 20220309-errros branch from a3934f8 to a9f5da5 Compare March 11, 2022 12:59
@knz
Copy link
Contributor Author

knz commented Mar 11, 2022

bors r=yuzefovich

This upgrades to v1.9.0 and enhances Sentry reports as requested by
the SQL queries team.

Release justification: low risk, high benefit changes to existing functionality

Release note: None
@craig
Copy link
Contributor

craig bot commented Mar 11, 2022

Build succeeded:

@craig craig bot merged commit 1477693 into cockroachdb:master Mar 11, 2022
@knz knz deleted the 20220309-errros branch March 11, 2022 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-server-and-security DB Server & Security
Projects
None yet
Development

Successfully merging this pull request may close these issues.

sentry: make sentry events more useful
3 participants