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 <[email protected]>
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 <[email protected]>
---
.../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());