bf2343b7e4
Integration Tests - MySQL + Elasticsearch / Detect Changes (push) Has been cancelled
Integration Tests - MySQL + Elasticsearch / integration-tests-mysql-elasticsearch (push) Has been cancelled
Integration Tests - PostgreSQL + Elasticsearch + Redis / Detect Changes (push) Has been cancelled
Integration Tests - PostgreSQL + Elasticsearch + Redis / integration-tests-postgres-elasticsearch-redis (push) Has been cancelled
Integration Tests - PostgreSQL + OpenSearch / Detect Changes (push) Has been cancelled
Integration Tests - PostgreSQL + OpenSearch / integration-tests-postgres-opensearch (push) Has been cancelled
Java Checkstyle / java-checkstyle (push) Has been cancelled
Maven Collate Tests / maven-collate-ci (push) Has been cancelled
OpenMetadata Service Unit Tests / openmetadata-service-unit-tests-status (push) Has been cancelled
Publish Package to Maven Central Repository / publish-maven-packages (push) Has been cancelled
OpenMetadata Service Unit Tests / Detect Changes (push) Has been cancelled
OpenMetadata Service Unit Tests / openmetadata-service-unit-tests (push) Has been cancelled
OpenMetadata Service Unit Tests / k8s_operator-unit-tests (push) Has been cancelled
188 lines
7.4 KiB
Python
188 lines
7.4 KiB
Python
# Copyright 2025 Collate
|
|
# Licensed under the Collate Community License, Version 1.0 (the "License");
|
|
# you may not use this file except in compliance with the License.
|
|
# You may obtain a copy of the License at
|
|
# https://github.com/open-metadata/OpenMetadata/blob/main/ingestion/LICENSE
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
# distributed under the License is distributed on an "AS IS" BASIS,
|
|
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
# See the License for the specific language governing permissions and
|
|
# limitations under the License.
|
|
"""TopologyProgressTracker: gating, path resolution, handles, totals hook."""
|
|
|
|
from unittest.mock import MagicMock
|
|
|
|
import pytest
|
|
|
|
from metadata.ingestion.api.topology_runner import TopologyRunnerMixin
|
|
from metadata.ingestion.models.topology import TopologyContextManager, get_topology_node, get_topology_root
|
|
from metadata.ingestion.progress.modes import ProgressMode, TotalsDeclarer
|
|
from metadata.ingestion.progress.runner_tracker import (
|
|
NO_OP_NODE_PROGRESS,
|
|
NodeProgress,
|
|
TopologyProgressTracker,
|
|
)
|
|
from metadata.ingestion.source.database.database_service import DatabaseServiceTopology
|
|
|
|
|
|
class _FakeSource(TopologyRunnerMixin):
|
|
topology = DatabaseServiceTopology()
|
|
|
|
def __init__(self, mode=ProgressMode.AUTO):
|
|
self.progress_mode = mode
|
|
self.context = TopologyContextManager(self.topology)
|
|
self.status = MagicMock()
|
|
self.declared = 0
|
|
|
|
def declare_progress_totals(self, totals: TotalsDeclarer) -> None:
|
|
self.declared += 1
|
|
totals.set_total("Database", 7)
|
|
|
|
|
|
def _table_node(source):
|
|
return get_topology_node("table", source.topology)
|
|
|
|
|
|
def _database_node(source):
|
|
return get_topology_node("database", source.topology)
|
|
|
|
|
|
def test_for_node_returns_noop_when_mode_is_not_auto():
|
|
for mode in (ProgressMode.MANUAL, ProgressMode.OFF):
|
|
source = _FakeSource(mode)
|
|
tracker = TopologyProgressTracker(source)
|
|
assert tracker.for_node(_table_node(source), is_leaf=True) is NO_OP_NODE_PROGRESS
|
|
assert "_progress_tracking" not in source.__dict__
|
|
|
|
|
|
def test_for_node_returns_noop_for_root_node():
|
|
source = _FakeSource()
|
|
tracker = TopologyProgressTracker(source)
|
|
root = get_topology_root(source.topology)[0]
|
|
tracker.on_walk_start([root])
|
|
assert tracker.is_root_node(root)
|
|
assert tracker.for_node(root, is_leaf=False) is NO_OP_NODE_PROGRESS
|
|
|
|
|
|
def test_for_node_returns_lazy_handle_for_auto_leaf():
|
|
source = _FakeSource()
|
|
tracker = TopologyProgressTracker(source)
|
|
handle = tracker.for_node(_table_node(source), is_leaf=True)
|
|
assert isinstance(handle, NodeProgress)
|
|
assert handle.wants_eager_count is False
|
|
|
|
|
|
def test_totals_hook_called_exactly_once_and_only_for_real_nodes():
|
|
source = _FakeSource()
|
|
tracker = TopologyProgressTracker(source)
|
|
root = get_topology_root(source.topology)[0]
|
|
tracker.on_walk_start([root])
|
|
tracker.for_node(root, is_leaf=False)
|
|
assert source.declared == 0
|
|
tracker.for_node(_database_node(source), is_leaf=False)
|
|
tracker.for_node(_table_node(source), is_leaf=True)
|
|
assert source.declared == 1
|
|
assert ("Database", 0, 7) in source.progress_tracking.registry.global_counters()
|
|
|
|
|
|
class _RaisingTotalsSource(_FakeSource):
|
|
def declare_progress_totals(self, totals: TotalsDeclarer) -> None:
|
|
self.declared += 1
|
|
raise RuntimeError("totals query blew up")
|
|
|
|
|
|
def test_totals_hook_failure_does_not_abort_the_walk():
|
|
source = _RaisingTotalsSource()
|
|
tracker = TopologyProgressTracker(source)
|
|
handle = tracker.for_node(_database_node(source), is_leaf=False)
|
|
assert isinstance(handle, NodeProgress) # walk continues, no exception propagates
|
|
tracker.for_node(_table_node(source), is_leaf=True)
|
|
assert source.declared == 1 # marked declared up front, so it is not retried per node
|
|
|
|
|
|
def test_leaf_handle_counts_open_and_advance():
|
|
source = _FakeSource()
|
|
tracker = TopologyProgressTracker(source)
|
|
handle = tracker.for_node(_table_node(source), is_leaf=True)
|
|
handle.open(2)
|
|
handle.advance_leaf()
|
|
handle.advance_leaf()
|
|
snapshot = source.progress_tracking.registry.snapshot()
|
|
assert snapshot.child_type == "Table"
|
|
assert snapshot.processed == 2
|
|
|
|
|
|
def test_container_handle_is_lazy_without_reconcilable_counter():
|
|
source = _FakeSource()
|
|
tracker = TopologyProgressTracker(source)
|
|
handle = tracker.for_node(_database_node(source), is_leaf=False)
|
|
assert handle.wants_eager_count is False
|
|
handle.open(None)
|
|
assert source.progress_tracking.registry.snapshot().expected_by_type.get("Database") is None
|
|
|
|
|
|
def test_container_handle_reconciles_when_counter_is_reconcilable():
|
|
source = _FakeSource()
|
|
source.progress_tracking.registry.seed_scope_total("DatabaseSchema", "salesdb", 1)
|
|
tracker = TopologyProgressTracker(source)
|
|
key = source._node_primary_stage(_database_node(source)).context
|
|
setattr(source.context.get(), key, "salesdb")
|
|
schema_node = get_topology_node("databaseSchema", source.topology)
|
|
handle = tracker.for_node(schema_node, is_leaf=False)
|
|
assert handle.wants_eager_count is True
|
|
handle.open(3)
|
|
counters = {t: (done, total) for t, done, total in source.progress_tracking.registry.global_counters()}
|
|
assert counters["DatabaseSchema"] == (0, 3)
|
|
|
|
|
|
def test_enter_scope_closes_and_tracks_on_exit():
|
|
source = _FakeSource()
|
|
source.progress_tracking.registry.set_total("Database", 7)
|
|
tracker = TopologyProgressTracker(source)
|
|
node = _database_node(source)
|
|
handle = tracker.for_node(node, is_leaf=False)
|
|
key = source._node_primary_stage(node).context
|
|
setattr(source.context.get(), key, "salesdb")
|
|
closed = []
|
|
source.progress_tracking.registry.close = lambda path: closed.append(list(path))
|
|
with handle.enter_scope():
|
|
pass
|
|
assert closed == [["salesdb"]]
|
|
counters = {t: (done, total) for t, done, total in source.progress_tracking.registry.global_counters()}
|
|
assert counters["Database"][0] == 1
|
|
|
|
|
|
def test_enter_scope_prunes_without_counting_on_failure():
|
|
source = _FakeSource()
|
|
source.progress_tracking.registry.set_total("Database", 7)
|
|
tracker = TopologyProgressTracker(source)
|
|
node = _database_node(source)
|
|
handle = tracker.for_node(node, is_leaf=False)
|
|
key = source._node_primary_stage(node).context
|
|
setattr(source.context.get(), key, "salesdb")
|
|
closed = []
|
|
source.progress_tracking.registry.close = lambda path: closed.append(list(path))
|
|
with pytest.raises(RuntimeError), handle.enter_scope():
|
|
raise RuntimeError("boom")
|
|
assert closed == [["salesdb"]]
|
|
counters = {t: (done, total) for t, done, total in source.progress_tracking.registry.global_counters()}
|
|
assert counters["Database"][0] == 0
|
|
|
|
|
|
def test_enter_scope_is_noop_without_context_value():
|
|
source = _FakeSource()
|
|
tracker = TopologyProgressTracker(source)
|
|
handle = tracker.for_node(_database_node(source), is_leaf=False)
|
|
with handle.enter_scope():
|
|
pass
|
|
assert source.progress_tracking.registry.snapshot() is None # nothing opened, nothing closed
|
|
|
|
|
|
def test_current_path_resolves_ancestor_context():
|
|
source = _FakeSource()
|
|
tracker = TopologyProgressTracker(source)
|
|
assert tracker.current_path() == []
|
|
db_key = source._node_primary_stage(_database_node(source)).context
|
|
setattr(source.context.get(), db_key, "salesdb")
|
|
assert tracker.current_path("DatabaseSchema") == ["salesdb"]
|