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