Skip to content

Commit

Permalink
Adding support for entity health
Browse files Browse the repository at this point in the history
  • Loading branch information
John Joyce authored and John Joyce committed Jan 25, 2024
1 parent 8ad46cc commit c8e2733
Show file tree
Hide file tree
Showing 5 changed files with 17 additions and 5 deletions.
1 change: 1 addition & 0 deletions datahub-web-react/src/app/entity/chart/ChartEntity.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -238,6 +238,7 @@ export class ChartEntity implements Entity<Chart> {
icon: entity?.platform?.properties?.logoUrl || undefined,
platform: entity?.platform,
subtype: entity?.subTypes?.typeNames?.[0] || undefined,
health: entity?.health || undefined,
};
};

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -259,6 +259,7 @@ export class DashboardEntity implements Entity<Dashboard> {
subtype: entity?.subTypes?.typeNames?.[0] || undefined,
icon: entity?.platform?.properties?.logoUrl || undefined,
platform: entity?.platform,
health: entity?.health || undefined,
};
};

Expand Down
1 change: 1 addition & 0 deletions datahub-web-react/src/app/entity/dataJob/DataJobEntity.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -225,6 +225,7 @@ export class DataJobEntity implements Entity<DataJob> {
type: EntityType.DataJob,
icon: entity?.dataFlow?.platform?.properties?.logoUrl || undefined,
platform: entity?.dataFlow?.platform,
health: entity?.health || undefined,
};
};

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ export const EntityHealth = ({ health, baseUrl, fontSize, tooltipPlacement }: Pr
return (
<>
{(unhealthy && (
<Link to={`${baseUrl}/Validation`}>
<Link to={`${baseUrl}/Incidents`}>
<Container>
<EntityHealthPopover health={health} baseUrl={baseUrl} placement={tooltipPlacement}>
{icon}
Expand Down
17 changes: 13 additions & 4 deletions datahub-web-react/src/graphql/lineage.graphql
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,9 @@ fragment lineageNodeProperties on EntityWithRelationships {
}
}
}
health {
...entityHealth
}
}
... on DataFlow {
orchestrator
Expand Down Expand Up @@ -89,6 +92,9 @@ fragment lineageNodeProperties on EntityWithRelationships {
status {
removed
}
health {
...entityHealth
}
}
... on Dashboard {
urn
Expand Down Expand Up @@ -144,6 +150,9 @@ fragment lineageNodeProperties on EntityWithRelationships {
subTypes {
typeNames
}
health {
...entityHealth
}
}
... on Chart {
tool
Expand Down Expand Up @@ -174,6 +183,9 @@ fragment lineageNodeProperties on EntityWithRelationships {
subTypes {
typeNames
}
health {
...entityHealth
}
}
... on Dataset {
name
Expand Down Expand Up @@ -211,10 +223,7 @@ fragment lineageNodeProperties on EntityWithRelationships {
}
}
health {
type
status
message
causes
...entityHealth
}
}
... on MLModelGroup {
Expand Down

0 comments on commit c8e2733

Please sign in to comment.