From df229597b1fec72f7d101d99844267cf197db85c Mon Sep 17 00:00:00 2001 From: Pinaki Bhattacharjee Date: Sat, 3 Aug 2024 16:42:05 +0530 Subject: [PATCH 1/2] Comparison of narrow type with wide type in loop condition --- .../metadata/boot/steps/IngestDataPlatformInstancesStep.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/metadata-service/factories/src/main/java/com/linkedin/metadata/boot/steps/IngestDataPlatformInstancesStep.java b/metadata-service/factories/src/main/java/com/linkedin/metadata/boot/steps/IngestDataPlatformInstancesStep.java index 638d1a6fd0c8a..22ce06a5984ea 100644 --- a/metadata-service/factories/src/main/java/com/linkedin/metadata/boot/steps/IngestDataPlatformInstancesStep.java +++ b/metadata-service/factories/src/main/java/com/linkedin/metadata/boot/steps/IngestDataPlatformInstancesStep.java @@ -61,7 +61,7 @@ public void execute(@Nonnull OperationContext systemOperationContext) throws Exc long numEntities = _migrationsDao.countEntities(); int start = 0; - while (start < numEntities) { + while (start < (int) numEntities) { log.info( "Reading urns {} to {} from the aspects table to generate dataplatform instance aspects", start, From c89b97d5ac2344ad632834b7c5f84476ee41b0b0 Mon Sep 17 00:00:00 2001 From: Pinaki Bhattacharjee Date: Sat, 3 Aug 2024 16:59:51 +0530 Subject: [PATCH 2/2] update loop --- .../com/linkedin/metadata/models/extractor/FieldExtractor.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/entity-registry/src/main/java/com/linkedin/metadata/models/extractor/FieldExtractor.java b/entity-registry/src/main/java/com/linkedin/metadata/models/extractor/FieldExtractor.java index 899f66e66ea5a..bef7782d8f7c9 100644 --- a/entity-registry/src/main/java/com/linkedin/metadata/models/extractor/FieldExtractor.java +++ b/entity-registry/src/main/java/com/linkedin/metadata/models/extractor/FieldExtractor.java @@ -63,7 +63,7 @@ public static Map> extractFields( } else { List valueList = (List) value.get(); // If the field is a nested list of values, flatten it - for (int i = 0; i < numArrayWildcards - 1; i++) { + for (long i = 0; i < numArrayWildcards - 1; i++) { valueList = valueList.stream() .flatMap(v -> ((List) v).stream())