-
Notifications
You must be signed in to change notification settings - Fork 89
datastore: update postgres package
#1092
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
base: main
Are you sure you want to change the base?
Conversation
82e3e3c to
26b9e6d
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1092 +/- ##
==========================================
- Coverage 55.79% 54.32% -1.48%
==========================================
Files 265 249 -16
Lines 16552 19713 +3161
==========================================
+ Hits 9236 10709 +1473
- Misses 6357 8043 +1686
- Partials 959 961 +2 ☔ View full report in Codecov by Sentry. |
0cd0bb1 to
a08b31c
Compare
|
@crozzy regarding the big-boy "add new package" commit: any opinion on how to split it up? I was thinking the common code and associated tests in a few commits, then the matcher and indexer implementations and tests in separate commits. |
9b929c8 to
11e2f8c
Compare
95d1cd2 to
9d1f0ae
Compare
|
I think is is good to get reviewed. I think the tests need a later pass to work through the |
0499148 to
ca605cf
Compare
ca605cf to
3df8829
Compare
3df8829 to
0c1d0e2
Compare
0c1d0e2 to
c185028
Compare
c185028 to
d0a6758
Compare
d0a6758 to
a2c16b0
Compare
a2c16b0 to
88f6891
Compare
Signed-off-by: Hank Donnay <hdonnay@redhat.com>
Signed-off-by: Hank Donnay <hdonnay@redhat.com>
Signed-off-by: Hank Donnay <hdonnay@redhat.com>
Finally. This copies the interface out of the `updater` package and then adds a process-local implementation that's simpler than the one in that package. Signed-off-by: Hank Donnay <hdonnay@redhat.com>
Signed-off-by: Hank Donnay <hdonnay@redhat.com>
Signed-off-by: Hank Donnay <hdonnay@redhat.com>
This renames the `MatcherStore` to a versioned name, namespaces the relevant supporting types, and adds aliases to retain API compatibility. Signed-off-by: Hank Donnay <hdonnay@redhat.com>
Signed-off-by: Hank Donnay <hdonnay@redhat.com>
Signed-off-by: Hank Donnay <hdonnay@redhat.com>
Signed-off-by: Hank Donnay <hdonnay@redhat.com>
Signed-off-by: Hank Donnay <hdonnay@redhat.com>
Signed-off-by: Hank Donnay <hdonnay@redhat.com>
Signed-off-by: Hank Donnay <hdonnay@redhat.com>
Signed-off-by: Hank Donnay <hdonnay@redhat.com>
Signed-off-by: Hank Donnay <hdonnay@redhat.com>
Signed-off-by: Hank Donnay <hdonnay@redhat.com>
Signed-off-by: Hank Donnay <hdonnay@redhat.com>
88f6891 to
817d136
Compare
Signed-off-by: Hank Donnay <hdonnay@redhat.com>
817d136 to
123b1f6
Compare
These changes are the result of trying to go through and apply a uniform style to the
datastore/postgrespackage.TODO
ctxlock/v2to alocksourcehierarchy?(started)slicespackagedowngrade the(It works just fine to have Clair carry the Jaeger requirement and make MVS do the work.)otelpackages to 1.18 (for Clair reasons -- it needs Jaeger export support)