From 2152fd6766361633941c2829aff41a714398ca60 Mon Sep 17 00:00:00 2001 From: Gabor Kaszab Date: Fri, 5 May 2023 16:32:41 +0200 Subject: [PATCH] Core: Remove duplicate check for ManifestEntry.dataSequenceNumber() When dropping ManifestEntry.sequenceNumber in #6274, a new assert was added to validateManifest() and validateDeleteManifest(). However, there was a same assert already. Dropping one of them. --- core/src/test/java/org/apache/iceberg/TableTestBase.java | 8 -------- 1 file changed, 8 deletions(-) diff --git a/core/src/test/java/org/apache/iceberg/TableTestBase.java b/core/src/test/java/org/apache/iceberg/TableTestBase.java index c03fb8cbd777..c0149d96499a 100644 --- a/core/src/test/java/org/apache/iceberg/TableTestBase.java +++ b/core/src/test/java/org/apache/iceberg/TableTestBase.java @@ -508,10 +508,6 @@ void validateManifest( "Data sequence number should match expected", expectedSequenceNumber, entry.dataSequenceNumber()); - V2Assert.assertEquals( - "Sequence number should match expected", - expectedSequenceNumber, - entry.dataSequenceNumber()); } if (fileSeqs != null) { V1Assert.assertEquals( @@ -550,10 +546,6 @@ void validateDeleteManifest( "Data sequence number should match expected", expectedSequenceNumber, entry.dataSequenceNumber()); - V2Assert.assertEquals( - "Sequence number should match expected", - expectedSequenceNumber, - entry.dataSequenceNumber()); } if (fileSeqs != null) { V1Assert.assertEquals(