MinimumRecordCountValidator logging tweaks

Sunjeet · Sunjeet · commit eb0e58d48e4a · 2025-06-18T16:27:25.000-07:00
diff --git a/hollow/src/main/java/com/netflix/hollow/api/producer/validation/MinimumRecordCountValidator.java b/hollow/src/main/java/com/netflix/hollow/api/producer/validation/MinimumRecordCountValidator.java
@@ -26,7 +26,7 @@ public MinimumRecordCountValidator(String typeName, Supplier<Integer> minRecordC
 
     @Override
     public String getName() {
-        return MinimumRecordCountValidator.class.getName() + "_" + typeName;
+        return MinimumRecordCountValidator.class.getSimpleName() + "_" + typeName;
     }
 
     @Override
@@ -53,7 +53,7 @@ public ValidationResult onValidate(HollowProducer.ReadState readState) {
             return vrb.failed(message);
         }
 
-        return vrb.passed(getName() + " " + typeName + " has >= " + minRecordCount + " records");
+        return vrb.passed(getName() + " numRecords=" + recordCount + ", threshold=" + minRecordCount);
     }
 
 }

Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ public MinimumRecordCountValidator(String typeName, Supplier<Integer> minRecordC`
`26`	`26`
`27`	`27`	`@Override`
`28`	`28`	`public String getName() {`
`29`		`- return MinimumRecordCountValidator.class.getName() + "_" + typeName;`
	`29`	`+ return MinimumRecordCountValidator.class.getSimpleName() + "_" + typeName;`
`30`	`30`	`}`
`31`	`31`
`32`	`32`	`@Override`
`@@ -53,7 +53,7 @@ public ValidationResult onValidate(HollowProducer.ReadState readState) {`
`53`	`53`	`return vrb.failed(message);`
`54`	`54`	`}`
`55`	`55`
`56`		`- return vrb.passed(getName() + " " + typeName + " has >= " + minRecordCount + " records");`
	`56`	`+ return vrb.passed(getName() + " numRecords=" + recordCount + ", threshold=" + minRecordCount);`
`57`	`57`	`}`
`58`	`58`
`59`	`59`	`}`