Fix #2568 Show database/entity_name in Lineage Node Label. (#2744)

* Fix #2568 Show database/entity_name in Lineage Node Label.

* Changing `/` to `.` as separator.
This commit is contained in:
Sachin Chaurasiya 2022-02-15 12:21:20 +05:30 committed by GitHub
parent 5261c2bad8
commit 3f5c8b8d95
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 10 additions and 4 deletions

View File

@ -168,7 +168,7 @@ const Entitylineage: FunctionComponent<EntityLineageProp> = ({
) : null}
<p className="tw-flex">
<span className="tw-mr-2">{getEntityIcon(node.type)}</span>
{getDataLabel(node.displayName, node.name)}
{getDataLabel(node.displayName, node.name, '.', false, node.type)}
</p>
</>
);

View File

@ -80,7 +80,7 @@ const NodeSuggestions: FC<EntitySuggestionProps> = ({
});
}}>
{entity.database
? `${entity.database}/${entity.name}`
? `${entity.database}.${entity.name}`
: entity.name}
</span>
</div>

View File

@ -399,8 +399,10 @@ export const getDataLabel = (
displayName?: string,
name = '',
separator = '.',
isTextOnly = false
isTextOnly = false,
type?: string
) => {
const databaseName = getPartialNameFromFQN(name, ['database']);
let label = '';
if (displayName) {
label = displayName;
@ -417,7 +419,11 @@ export const getDataLabel = (
<span
className="tw-break-words description-text tw-self-center"
data-testid="lineage-entity">
{label}
{type === 'table'
? databaseName
? `${databaseName}.${label}`
: label
: label}
</span>
);
};