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

[Feature] Datacatalog cache deletion #4655

Open
wants to merge 69 commits into
base: master
Choose a base branch
from

merge master

9e322a8
Select commit
Loading
Failed to load commit list.
Open

[Feature] Datacatalog cache deletion #4655

merge master
9e322a8
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Mar 12, 2024 in 1s

67.56% of diff hit (target 59.00%)

View this Pull Request on Codecov

67.56% of diff hit (target 59.00%)

Annotations

Check warning on line 368 in datacatalog/pkg/manager/impl/artifact_manager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

datacatalog/pkg/manager/impl/artifact_manager.go#L368

Added line #L368 was not covered by tests

Check warning on line 441 in datacatalog/pkg/manager/impl/artifact_manager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

datacatalog/pkg/manager/impl/artifact_manager.go#L438-L441

Added lines #L438 - L441 were not covered by tests

Check warning on line 96 in datacatalog/pkg/manager/impl/reservation_manager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

datacatalog/pkg/manager/impl/reservation_manager.go#L94-L96

Added lines #L94 - L96 were not covered by tests

Check warning on line 200 in datacatalog/pkg/manager/impl/reservation_manager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

datacatalog/pkg/manager/impl/reservation_manager.go#L199-L200

Added lines #L199 - L200 were not covered by tests

Check warning on line 193 in datacatalog/pkg/repositories/gormimpl/artifact.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

datacatalog/pkg/repositories/gormimpl/artifact.go#L192-L193

Added lines #L192 - L193 were not covered by tests

Check warning on line 198 in datacatalog/pkg/repositories/gormimpl/artifact.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

datacatalog/pkg/repositories/gormimpl/artifact.go#L197-L198

Added lines #L197 - L198 were not covered by tests

Check warning on line 225 in datacatalog/pkg/repositories/gormimpl/artifact.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

datacatalog/pkg/repositories/gormimpl/artifact.go#L223-L225

Added lines #L223 - L225 were not covered by tests

Check warning on line 247 in datacatalog/pkg/repositories/gormimpl/artifact.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

datacatalog/pkg/repositories/gormimpl/artifact.go#L246-L247

Added lines #L246 - L247 were not covered by tests

Check warning on line 52 in flyteadmin/pkg/manager/impl/cache_manager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/manager/impl/cache_manager.go#L50-L52

Added lines #L50 - L52 were not covered by tests

Check warning on line 89 in flyteadmin/pkg/manager/impl/cache_manager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/manager/impl/cache_manager.go#L71-L89

Added lines #L71 - L89 were not covered by tests

Check warning on line 107 in flyteadmin/pkg/manager/impl/cache_manager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/manager/impl/cache_manager.go#L93-L107

Added lines #L93 - L107 were not covered by tests

Check warning on line 127 in flyteadmin/pkg/manager/impl/cache_manager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/manager/impl/cache_manager.go#L113-L127

Added lines #L113 - L127 were not covered by tests

Check warning on line 247 in flyteadmin/pkg/manager/impl/cache_manager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/manager/impl/cache_manager.go#L235-L247

Added lines #L235 - L247 were not covered by tests

Check warning on line 22 in flytestdlib/catalog/client.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytestdlib/catalog/client.go#L13-L22

Added lines #L13 - L22 were not covered by tests

Check warning on line 24 in flytestdlib/catalog/client.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytestdlib/catalog/client.go#L24

Added line #L24 was not covered by tests

Check warning on line 473 in flytestdlib/catalog/datacatalog/datacatalog.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytestdlib/catalog/datacatalog/datacatalog.go#L472-L473

Added lines #L472 - L473 were not covered by tests

Check warning on line 480 in flytestdlib/catalog/datacatalog/datacatalog.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytestdlib/catalog/datacatalog/datacatalog.go#L479-L480

Added lines #L479 - L480 were not covered by tests

Check warning on line 161 in flytestdlib/catalog/datacatalog/transformer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytestdlib/catalog/datacatalog/transformer.go#L152-L161

Added lines #L152 - L161 were not covered by tests

Check warning on line 39 in flytestdlib/catalog/noop_catalog.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytestdlib/catalog/noop_catalog.go#L38-L39

Added lines #L38 - L39 were not covered by tests

Check warning on line 47 in flytestdlib/catalog/noop_catalog.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytestdlib/catalog/noop_catalog.go#L46-L47

Added lines #L46 - L47 were not covered by tests

Check warning on line 51 in flytestdlib/catalog/noop_catalog.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytestdlib/catalog/noop_catalog.go#L50-L51

Added lines #L50 - L51 were not covered by tests

Check warning on line 55 in flytestdlib/catalog/noop_catalog.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytestdlib/catalog/noop_catalog.go#L54-L55

Added lines #L54 - L55 were not covered by tests

Check warning on line 59 in flytestdlib/catalog/noop_catalog.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytestdlib/catalog/noop_catalog.go#L58-L59

Added lines #L58 - L59 were not covered by tests