This is an automated email from the ASF dual-hosted git repository. dlmarion pushed a commit to branch elasticity in repository https://gitbox.apache.org/repos/asf/accumulo.git
The following commit(s) were added to refs/heads/elasticity by this push: new 637dd0fd3f Removed elasticity comment to do in ExternalCompaction_2_IT (#4416) 637dd0fd3f is described below commit 637dd0fd3f4ce9e408b93db59816e1b2b07b6a9e Author: Dave Marion <dlmar...@apache.org> AuthorDate: Fri Apr 5 10:11:25 2024 -0400 Removed elasticity comment to do in ExternalCompaction_2_IT (#4416) The comment said that the operation id needed to be set when deleting the tablets. This is now done in ReserveTablets.isReady. Co-authored-by: Keith Turner <ktur...@apache.org> --- .../org/apache/accumulo/test/compaction/ExternalCompaction_2_IT.java | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/test/src/main/java/org/apache/accumulo/test/compaction/ExternalCompaction_2_IT.java b/test/src/main/java/org/apache/accumulo/test/compaction/ExternalCompaction_2_IT.java index 937cb03037..4ca3794ca9 100644 --- a/test/src/main/java/org/apache/accumulo/test/compaction/ExternalCompaction_2_IT.java +++ b/test/src/main/java/org/apache/accumulo/test/compaction/ExternalCompaction_2_IT.java @@ -238,9 +238,7 @@ public class ExternalCompaction_2_IT extends SharedMiniClusterBase { confirmCompactionCompleted(getCluster().getServerContext(), ecids, TCompactionState.CANCELLED); - // ELASTICITY_TODO make delete table fate op get operation ids before deleting - // there should be no metadata for the table, check to see if the compaction wrote anything - // after table delete + // Ensure compaction did not write anything to metadata table after delete table try (var scanner = client.createScanner(AccumuloTable.METADATA.tableName())) { scanner.setRange(MetadataSchema.TabletsSection.getRange(tid)); assertEquals(0, scanner.stream().count());