Skip to content

Commit 130e58d

Browse files
committed
Fix conflict
Signed-off-by: Terence <terencelimxp@gmail.com>
1 parent b1f0e5c commit 130e58d

1 file changed

Lines changed: 3 additions & 15 deletions

File tree

serving/src/main/java/feast/serving/service/OnlineServingServiceV2.java

Lines changed: 3 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -161,22 +161,10 @@ public GetOnlineFeaturesResponse getOnlineFeatures(GetOnlineFeaturesRequestV2 re
161161
populateCountMetrics(statusMap, projectName);
162162
}
163163
}
164-
populateHistogramMetrics(entityRows, featureReferences, projectName);
165-
166-
// Build response field values from entityValuesMap and entityStatusesMap
167-
// Response field values should be in the same order as the entityRows provided by the user.
168-
List<GetOnlineFeaturesResponse.FieldValues> fieldValuesList =
169-
entityRows.stream()
170-
.map(
171-
entityRow -> {
172-
return GetOnlineFeaturesResponse.FieldValues.newBuilder()
173-
.putAllFields(entityValuesMap.get(entityRow))
174-
.putAllStatuses(entityStatusesMap.get(entityRow))
175-
.build();
176-
})
177-
.collect(Collectors.toList());
178-
return GetOnlineFeaturesResponse.newBuilder().addAllFieldValues(fieldValuesList).build();
164+
entityValuesMap.get(entityRow).putAll(allValueMaps);
165+
entityStatusesMap.get(entityRow).putAll(allStatusMaps);
179166
}
167+
populateHistogramMetrics(entityRows, featureReferences, projectName);
180168

181169
// Build response field values from entityValuesMap and entityStatusesMap
182170
// Response field values should be in the same order as the entityRows provided by the user.

0 commit comments

Comments
 (0)