-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
use sqlutil instead of pg.QOpts/Q/Queryer #12456
Conversation
1ea946d
to
3f25e2f
Compare
I see that you haven't updated any README files. Would it make sense to do so? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This tx was not actually used.
3f25e2f
to
e2ffa4d
Compare
e2ffa4d
to
8d5a4c8
Compare
8d5a4c8
to
33dd172
Compare
33dd172
to
106265b
Compare
106265b
to
d56b016
Compare
d56b016
to
8d69cd8
Compare
I see you updated files related to
|
c78844c
to
94c4e8e
Compare
Why do you say |
Quality Gate passedIssues Measures |
https://smartcontract-it.atlassian.net/browse/BCF-2978
Remove
pg.QOpts
,pg.Q
/pg.Queryer
, and related helpers. Usepackage sqlutil
from the comon repo.The
sqlutil.DataStore
interface only containscontext.Context
methods, so as of this change 99% of our database queries have acontext.Context
wired through to support tracing via open telemetry.Requires:
TODO: