Skip to content

Commit 980f791

Browse files
committed
removal of try-catch and rewriting log statements
1 parent 74eaa30 commit 980f791

File tree

2 files changed

+3
-11
lines changed

2 files changed

+3
-11
lines changed

service/src/main/java/uk/nhs/adaptors/gp2gp/ehr/EhrExtractStatusService.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -440,7 +440,7 @@ private void getEhrExtractStatus(
440440
Update.AddToSetBuilder updateBuilder,
441441
FindAndModifyOptions returningUpdatedRecordOption) {
442442

443-
LOGGER.debug("Attempting to update EHR extract status. Query: {}, Number of documents: {}", query, docs.size());
443+
LOGGER.debug("Attempting to update EHR extract status. Number of documents: {}", docs.size());
444444

445445
Update update = updateBuilder.each(docs);
446446

service/src/main/java/uk/nhs/adaptors/gp2gp/ehr/SendEhrExtractCoreTaskExecutor.java

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -157,13 +157,9 @@ private void uploadCompressedEhrExtractToStorageWrapper(
157157
String taskId,
158158
String fileName) throws JsonProcessingException {
159159

160-
LOGGER.info("Preparing to upload compressed EHR extract. File name: {}, Document ID: {}, Task ID: {}, Message ID: {}",
161-
fileName, documentId, taskId, messageId);
160+
LOGGER.info("Preparing to upload compressed EHR extract. File name: {}, Message ID: {}", fileName, messageId);
162161

163-
String data;
164-
165-
try {
166-
data = objectMapper.writeValueAsString(
162+
String data = objectMapper.writeValueAsString(
167163
OutboundMessage.builder()
168164
.payload(
169165
ehrDocumentMapper.generateMhsPayload(
@@ -181,10 +177,6 @@ private void uploadCompressedEhrExtractToStorageWrapper(
181177
.payload(compressedEhrExtract)
182178
.build()
183179
)).build());
184-
} catch (JsonProcessingException exception) {
185-
LOGGER.error("Failed to serialize OutboundMessage for document ID: {}", documentId, exception);
186-
throw exception;
187-
}
188180

189181
storageConnectorService.uploadFile(
190182
StorageDataWrapperProvider.buildStorageDataWrapper(sendEhrExtractCoreTaskDefinition, data, taskId),

0 commit comments

Comments
 (0)