Skip to content

Commit afa2bc7

Browse files
committed
Address comments
1 parent 42b2f8c commit afa2bc7

File tree

2 files changed

+7
-2
lines changed

2 files changed

+7
-2
lines changed

openmetadata-integration-tests/src/test/java/org/openmetadata/it/tests/PipelineResourceIT.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1299,6 +1299,7 @@ void put_pipelineStatusReindexesDownstreamLineage_200_OK(TestNamespace ns) throw
12991299
assertEquals(StatusType.Successful, updatedPipeline.getPipelineStatus().getExecutionStatus());
13001300
assertEquals(pipelineStatus.getTimestamp(), updatedPipeline.getPipelineStatus().getTimestamp());
13011301
}
1302+
13021303
// ===================================================================
13031304
// BULK API SUPPORT
13041305
// ===================================================================

openmetadata-service/src/main/java/org/openmetadata/service/jdbi3/PipelineRepository.java

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -388,7 +388,7 @@ public RestUtil.PutResponse<?> addBulkPipelineStatus(
388388

389389
Set<String> validatedTasks = new HashSet<>();
390390
for (PipelineStatus pipelineStatus : pipelineStatuses) {
391-
for (Status taskStatus : pipelineStatus.getTaskStatus()) {
391+
for (Status taskStatus : listOrEmpty(pipelineStatus.getTaskStatus())) {
392392
if (validatedTasks.add(taskStatus.getName())) {
393393
validateTask(pipeline, taskStatus.getName());
394394
}
@@ -416,7 +416,11 @@ public RestUtil.PutResponse<?> addBulkPipelineStatus(
416416
pipeline.setChangeDescription(change);
417417
pipeline.setIncrementalChangeDescription(change);
418418

419-
searchRepository.updateEntityIndex(pipeline);
419+
try {
420+
searchRepository.updateEntityIndex(pipeline);
421+
} catch (Exception e) {
422+
LOG.error("Failed to update pipeline entity index in Elasticsearch", e);
423+
}
420424

421425
return new RestUtil.PutResponse<>(
422426
Response.Status.OK,

0 commit comments

Comments
 (0)