mirror of
https://github.com/sourcegraph/sourcegraph.git
synced 2026-02-06 17:51:57 +00:00
9145768648
- The `internal/uploadstore` package is renamed to `object` indicating
that it is meant to provide a generic object storage wrapper.
- The `search/exhaustive/uploadstore` package is used in very few places
so I've merged into the `internal/search` package similar to
`internal/embeddings`.
There are a few reasons to do the renaming.
1. The word `upload` in a more general context is ambiguous (just in
`internal/`) - in the codeintel context, it means "SCIP index" but it
can also be interpreted generically ("upload of _some_ data").
2. Better readability - `object.Storage` is much shorter than
`uploadstore.Store`. Additionally, we use the term `Store` A LOT
in the codebase, and usually, these refer to wrappers over some
tables in some DB.
Making things worse, some of our code also has:
```
uploadsstore
"github.com/sourcegraph/sourcegraph/internal/codeintel/uploads/internal/store"
```
And code which says `uploadsstore.Store` (notice the extra `s` 😢), which
is actually a wrapper over some key DB tables like `lsif_uploads`.
61 lines
1.6 KiB
Python
61 lines
1.6 KiB
Python
load("//dev:go_mockgen.bzl", "go_mockgen")
|
|
load("//dev:go_defs.bzl", "go_test")
|
|
load("@io_bazel_rules_go//go:def.bzl", "go_library")
|
|
|
|
go_library(
|
|
name = "uploadhandler",
|
|
srcs = [
|
|
"iface.go",
|
|
"observability.go",
|
|
"upload_enqueuer.go",
|
|
"upload_enqueuer_observability.go",
|
|
"upload_handler.go",
|
|
"upload_handler_multipart.go",
|
|
"upload_handler_single.go",
|
|
"upload_handler_state.go",
|
|
],
|
|
importpath = "github.com/sourcegraph/sourcegraph/internal/uploadhandler",
|
|
tags = [TAG_PLATFORM_GRAPH],
|
|
visibility = ["//:__subpackages__"],
|
|
deps = [
|
|
"//internal/metrics",
|
|
"//internal/object",
|
|
"//internal/observation",
|
|
"//internal/trace/policy",
|
|
"//lib/errors",
|
|
"@com_github_aws_aws_sdk_go_v2_feature_s3_manager//:manager",
|
|
"@com_github_sourcegraph_log//:log",
|
|
"@io_opentelemetry_go_otel//attribute",
|
|
],
|
|
)
|
|
|
|
go_test(
|
|
name = "uploadhandler_test",
|
|
timeout = "short",
|
|
srcs = [
|
|
"mocks_test.go",
|
|
"upload_handler_test.go",
|
|
],
|
|
embed = [":uploadhandler"],
|
|
tags = [TAG_PLATFORM_GRAPH],
|
|
deps = [
|
|
"//internal/object",
|
|
"//internal/object/mocks",
|
|
"//internal/observation",
|
|
"@com_github_google_go_cmp//cmp",
|
|
"@com_github_sourcegraph_log//:log",
|
|
"@com_github_sourcegraph_log//logtest",
|
|
],
|
|
)
|
|
|
|
go_mockgen(
|
|
name = "generate_mocks",
|
|
out = "mocks_test.go",
|
|
manifests = [
|
|
"//:mockgen.yaml",
|
|
"//:mockgen.test.yaml",
|
|
"//:mockgen.temp.yaml",
|
|
],
|
|
deps = [":uploadhandler"],
|
|
)
|