Skip to content

Commit 1d01e8d

Browse files
committed
MAT-8638: Adjust async validation logging entries to have a consistent prefix to make downstream parsing easier.
Remove threadId from log entries, it was not helpful. Signed-off-by: Joseph Kotanchik <joseph.kotanchik@semanticbits.com>
1 parent e12938f commit 1d01e8d

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

src/main/java/cms/gov/madie/measure/services/TestCaseValidationService.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -102,9 +102,8 @@ void validate(
102102
// TODO replace with decorator
103103
Instant startTime = Instant.now();
104104
log.info(
105-
"TestCase Validation::execute::{}::{}::{}::{}",
105+
"TestCase Validation::execute::{}::{}::{}",
106106
submittedTestCase.getId(),
107-
Thread.currentThread().getId(),
108107
taskId,
109108
startTime);
110109
Measure measure =
@@ -133,9 +132,10 @@ void validate(
133132
currentTestCase.getId(), measureId, taskId, validationOutcome);
134133
Instant stopTime = Instant.now();
135134
log.info(
136-
"TestCase Validation::completed::{}::{}::{}::{}",
135+
"TestCase Validation::completed::{}::{}::{}::{}::{}",
137136
currentTestCase.getId(),
138137
taskId,
138+
stopTime,
139139
Duration.between(startTime, stopTime),
140140
saveExecutor.getQueueSize());
141141
} catch (Exception e) {
@@ -157,7 +157,7 @@ public TestCase validateResourceAsynchronously(
157157
// If the measure is null, the test case has already has PENDING status.
158158
if (updatedMeasure == null) {
159159
log.info(
160-
"Test Case with Id {} already in validation queue for Measure with Id {}",
160+
"TestCase Validation::already pending::{}::measure::{}",
161161
testCase.getId(),
162162
measure.getId());
163163
return testCase;

0 commit comments

Comments
 (0)