-
Notifications
You must be signed in to change notification settings - Fork 84
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
datastore: update postgres
package
#1092
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 <[email protected]>
Signed-off-by: Hank Donnay <[email protected]>
Signed-off-by: Hank Donnay <[email protected]>
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 <[email protected]>
Signed-off-by: Hank Donnay <[email protected]>
Signed-off-by: Hank Donnay <[email protected]>
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 <[email protected]>
Signed-off-by: Hank Donnay <[email protected]>
Signed-off-by: Hank Donnay <[email protected]>
Signed-off-by: Hank Donnay <[email protected]>
Signed-off-by: Hank Donnay <[email protected]>
Signed-off-by: Hank Donnay <[email protected]>
Signed-off-by: Hank Donnay <[email protected]>
Signed-off-by: Hank Donnay <[email protected]>
Signed-off-by: Hank Donnay <[email protected]>
Signed-off-by: Hank Donnay <[email protected]>
Signed-off-by: Hank Donnay <[email protected]>
88f6891
to
817d136
Compare
Signed-off-by: Hank Donnay <[email protected]>
817d136
to
123b1f6
Compare
These changes are the result of trying to go through and apply a uniform style to the
datastore/postgres
package.TODO
ctxlock/v2
to alocksource
hierarchy?(started)slices
packagedowngrade the(It works just fine to have Clair carry the Jaeger requirement and make MVS do the work.)otel
packages to 1.18 (for Clair reasons -- it needs Jaeger export support)