Qua 1882 datahub integration#1128
Conversation
Greptile SummaryThis PR refreshes the DataHub integration guide to match the current implementation and cleans up duplicate navigation entries in
Confidence Score: 5/5This PR is safe to merge — all changes are documentation and navigation only, with no code execution or functional logic involved. The datahub.md rewrite is internally consistent: the three-surface push model (Documentation, Structured Properties, Incidents) is described accurately and cross-referenced correctly across all sections. The mkdocs.yml navigation cleanup correctly removes two duplicate nav blocks while keeping the full top-level Qualytics CLI section and all 5 SSO/Compute/Workflow/Analytics pages reachable under Settings → Integrations. All five new screenshots are referenced in the markdown and present in the diff. No pages are orphaned or made unreachable. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[Qualytics Sync Triggered\nManual or Event Driven] --> B{Match Asset\nby qualified name}
B -->|Datastore| C[DataHub Container\nsubType: Schema]
B -->|Container / Table| D[DataHub Dataset]
B -->|Field / Column| E[DataHub SchemaField]
C --> F[Documentation Tab\nQuality summary + View in Qualytics link]
D --> F
D --> G[Structured Properties\n12 typed fields]
D --> H{Active Anomalies\n> 0?}
E --> F
E --> G
H -->|Yes| I[Raise Incident\nQualytics: Active Anomalies]
H -->|No| J{Incident exists?}
J -->|Yes| K[Resolve Incident\nautomatically ~10s]
J -->|No| L[No action]
M[DataHub Tags] -->|Pull via manual sync| N[Qualytics External Tags]
Reviews (3): Last reviewed commit: "Address greptile feedback" | Re-trigger Greptile |
|
@greptile review |
|
@greptile review |
Summary
Refresh the DataHub integration guide to match the current implementation, plus a small navigation cleanup.
DataHub documentation updates (
docs/settings/integrations/data-catalogs/datahub.md):Incidents tab).
Schema(not Platform/Dataset), with parent-database disambiguation. Added atip for instances with same-named schemas across databases.
View in Qualytics) instead of internal qualified names.
Navigation cleanup (
mkdocs.yml):refactor. Removed the duplicate top-level Integrations entry.
tree) remains as the single source of truth.