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());

Reply via email to