diff --git a/hollow/src/main/java/com/netflix/hollow/api/producer/validation/RecordCountVarianceValidator.java b/hollow/src/main/java/com/netflix/hollow/api/producer/validation/RecordCountVarianceValidator.java index e1426d4542..6f5fb0f530 100644 --- a/hollow/src/main/java/com/netflix/hollow/api/producer/validation/RecordCountVarianceValidator.java +++ b/hollow/src/main/java/com/netflix/hollow/api/producer/validation/RecordCountVarianceValidator.java @@ -61,8 +61,9 @@ public void validate(ReadState readState) { if (Float.compare(actualChangePercent, allowableVariancePercent) > 0) { throw new ValidationException("RecordCountVarianceValidator for type " + typeName + " failed. Actual variance: " + actualChangePercent + "%; Allowed variance: " - + allowableVariancePercent + "%; current cycle record count: " + latestCardinality - + "; previous cycle record count: " + previousCardinality); + + allowableVariancePercent + "%; previous cycle record count: " + previousCardinality + + "; current cycle record count: " + latestCardinality); + } } } diff --git a/hollow/src/test/java/com/netflix/hollow/api/producer/validation/RecordCountVarianceValidatorTests.java b/hollow/src/test/java/com/netflix/hollow/api/producer/validation/RecordCountVarianceValidatorTests.java index a3594cfe5b..9cb7d6d024 100644 --- a/hollow/src/test/java/com/netflix/hollow/api/producer/validation/RecordCountVarianceValidatorTests.java +++ b/hollow/src/test/java/com/netflix/hollow/api/producer/validation/RecordCountVarianceValidatorTests.java @@ -66,8 +66,7 @@ public void populate(WriteState newState) throws Exception { Assert.fail(); } catch (ValidationException expected) { Assert.assertEquals(1, expected.getIndividualFailures().size()); - // System.out.println("Message: - // "+expected.getIndividualFailures().get(0).getMessage()); + System.out.println("Message:"+expected.getIndividualFailures().get(0).getMessage()); Assert.assertTrue(expected.getIndividualFailures().get(0).getMessage() .startsWith("RecordCountVarianceValidator for type TypeWithPrimaryKey failed.")); } @@ -96,9 +95,8 @@ public void populate(WriteState newState) throws Exception { }); Assert.fail(); } catch (ValidationException expected) { + System.out.println("Message:"+expected.getIndividualFailures().get(0).getMessage()); Assert.assertEquals(1, expected.getIndividualFailures().size()); - // System.out.println("Message: - // "+expected.getIndividualFailures().get(0).getMessage()); Assert.assertTrue(expected.getIndividualFailures().get(0).getMessage() .startsWith("RecordCountVarianceValidator for type TypeWithPrimaryKey failed.")); }