You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by kt...@apache.org on 2023/11/11 20:20:14 UTC

(accumulo) branch main updated: fixes validation in MetaSplitIT (#3944)

This is an automated email from the ASF dual-hosted git repository.

kturner pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/accumulo.git


The following commit(s) were added to refs/heads/main by this push:
     new 71f1b0fbc4 fixes validation in MetaSplitIT (#3944)
71f1b0fbc4 is described below

commit 71f1b0fbc494342b5b659ed607ca67d5339848d1
Author: Keith Turner <kt...@apache.org>
AuthorDate: Sat Nov 11 15:20:07 2023 -0500

    fixes validation in MetaSplitIT (#3944)
    
    As part of validating the metadata table MetaSplitIT was requiring
    tablets to have a location.  However locations may be temproarily absent
    and that is ok.  Changed the test to validate the time field was present
    as it should always be there.
---
 test/src/main/java/org/apache/accumulo/test/MetaSplitIT.java | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/test/src/main/java/org/apache/accumulo/test/MetaSplitIT.java b/test/src/main/java/org/apache/accumulo/test/MetaSplitIT.java
index 0753463639..aeee11e9e9 100644
--- a/test/src/main/java/org/apache/accumulo/test/MetaSplitIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/MetaSplitIT.java
@@ -189,8 +189,9 @@ public class MetaSplitIT extends AccumuloClusterHarness {
     try (var tablets = ample.readTablets().forLevel(Ample.DataLevel.USER).build()) {
       for (var tablet : tablets) {
         assertTrue(expectedExtents.remove(tablet.getExtent()));
+        // check a few fields that should always be present in tablet metadata
         assertNotNull(tablet.getDirName());
-        assertNotNull(tablet.getLocation());
+        assertNotNull(tablet.getTime());
       }
     }