Skip to content

Commit f5f3ac2

Browse files
committed
Rename "history" to "artifacts"
1 parent cbaba94 commit f5f3ac2

File tree

4 files changed

+10
-10
lines changed

4 files changed

+10
-10
lines changed

src/databricks/labs/ucx/cli.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -611,7 +611,7 @@ def create_ucx_catalog(w: WorkspaceClient, prompts: Prompts, ctx: WorkspaceConte
611611
"""
612612
workspace_context = ctx or WorkspaceContext(w)
613613
workspace_context.catalog_schema.create_ucx_catalog(prompts)
614-
workspace_context.history_installation.run()
614+
workspace_context.artifact_installation.run()
615615

616616

617617
@ucx.command

src/databricks/labs/ucx/contexts/workspace_cli.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
from databricks.labs.ucx.azure.resources import AzureAPIClient, AzureResources
1919
from databricks.labs.ucx.contexts.application import CliContext
2020
from databricks.labs.ucx.hive_metastore.table_migration_status import TableMigrationIndex
21-
from databricks.labs.ucx.progress.install import HistoryInstallation
21+
from databricks.labs.ucx.progress.install import ArtifactInstallation
2222
from databricks.labs.ucx.source_code.base import CurrentSessionState
2323
from databricks.labs.ucx.source_code.linters.context import LinterContext
2424
from databricks.labs.ucx.source_code.linters.files import LocalFileMigrator, LocalCodeLinter
@@ -181,8 +181,8 @@ def notebook_loader(self) -> NotebookLoader:
181181
return NotebookLoader()
182182

183183
@cached_property
184-
def history_installation(self) -> HistoryInstallation:
185-
return HistoryInstallation(self.sql_backend, self.config.ucx_catalog)
184+
def artifact_installation(self) -> ArtifactInstallation:
185+
return ArtifactInstallation(self.sql_backend, self.config.ucx_catalog)
186186

187187

188188
class LocalCheckoutContext(WorkspaceContext):

src/databricks/labs/ucx/progress/install.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,10 +36,10 @@ class WorkflowRun:
3636
"""The workflow run final status"""
3737

3838

39-
class HistoryInstallation:
40-
"""Install resources for UCX's artifacts history."""
39+
class ArtifactInstallation:
40+
"""Install resources for UCX's artifacts."""
4141

42-
_SCHEMA = "history"
42+
_SCHEMA = "artifacts"
4343

4444
def __init__(self, sql_backend: SqlBackend, ucx_catalog: str) -> None:
4545
# `mod` is required parameter, but it's not used in this context.

tests/unit/progress/test_install.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,14 @@
1-
from databricks.labs.ucx.progress.install import HistoryInstallation
1+
from databricks.labs.ucx.progress.install import ArtifactInstallation
22

33

44
def test_history_installation_run_creates_history_schema(mock_backend) -> None:
5-
installation = HistoryInstallation(mock_backend, "ucx")
5+
installation = ArtifactInstallation(mock_backend, "ucx")
66
installation.run()
77
assert "CREATE SCHEMA IF NOT EXISTS ucx.history" in mock_backend.queries[0]
88

99

1010
def test_history_installation_run_creates_workflow_runs_table(mock_backend) -> None:
11-
installation = HistoryInstallation(mock_backend, "ucx")
11+
installation = ArtifactInstallation(mock_backend, "ucx")
1212
installation.run()
1313
# Dataclass to schema conversion is tested within the lsql package
1414
assert any("CREATE TABLE IF NOT EXISTS" in query for query in mock_backend.queries)

0 commit comments

Comments
 (0)