yujunjun/OpenMetadata
1
0
Fork 0
mirror of https://github.com/open-metadata/OpenMetadata.git synced 2026-01-07 13:07:22 +00:00
Code Issues Packages Projects Releases Wiki Activity

correct negative values of the changeCount. (#18279)

Browse Source
This commit is contained in:
Siddhant 2024-10-16 23:21:00 +05:30 committed by GitHub
parent 7012e73d75
commit 42eea35aff
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
openmetadata-service/src/main/java/org/openmetadata/service/apps/bundles/insights/DataInsightsReportApp.java
View File

@ -218,7 +218,7 @@ public class DataInsightsReportApp extends AbstractNativeApplication {
dateWithCount.forEach((key, value) -> dateMap.put(key, value.intValue()));
processDateMapToNormalize(dateMap);
int changeInTotalAssets = (int) (currentCount - previousCount);
int changeInTotalAssets = (int) Math.abs(currentCount - previousCount);
if (previousCount == 0D) {
// it should be undefined
@ -276,7 +276,7 @@ public class DataInsightsReportApp extends AbstractNativeApplication {
currentPercentCompleted = (currentCompletedDescription / currentTotalAssetCount) * 100;
}
int changeCount = (int) (currentCompletedDescription - previousCompletedDescription);
int changeCount = (int) Math.abs(currentCompletedDescription - previousCompletedDescription);
return getTemplate(
DataInsightDescriptionAndOwnerTemplate.MetricType.DESCRIPTION,
@ -327,7 +327,7 @@ public class DataInsightsReportApp extends AbstractNativeApplication {
currentPercentCompleted = (currentHasOwner / currentTotalAssetCount) * 100;
}
int changeCount = (int) (currentHasOwner - previousHasOwner);
int changeCount = (int) Math.abs(currentHasOwner - previousHasOwner);
return getTemplate(
DataInsightDescriptionAndOwnerTemplate.MetricType.OWNER,
@ -376,7 +376,7 @@ public class DataInsightsReportApp extends AbstractNativeApplication {
currentPercentCompleted = (currentHasTier / currentTotalAssetCount) * 100;
}
int changeCount = (int) (currentHasTier - previousHasTier);
int changeCount = (int) Math.abs(currentHasTier - previousHasTier);
// TODO: Understand if we actually use this tierData for anything.
Map<String, Double> tierData = new HashMap<>();