This is an automated email from the ASF dual-hosted git repository.
cshannon 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 cfe5738f8f Minor fix for naming of method/comment in
MetadataConstraints
cfe5738f8f is described below
commit cfe5738f8f7f2be800a24fead4c8413b054780fc
Author: Christopher L. Shannon (cshannon) <[email protected]>
AuthorDate: Sat Jul 15 11:41:22 2023 -0400
Minor fix for naming of method/comment in MetadataConstraints
Fixes the names for a method and in a comment that was added in PR #3506
---
.../accumulo/server/constraints/MetadataConstraints.java | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git
a/server/base/src/main/java/org/apache/accumulo/server/constraints/MetadataConstraints.java
b/server/base/src/main/java/org/apache/accumulo/server/constraints/MetadataConstraints.java
index 921639bda8..622e58cfc8 100644
---
a/server/base/src/main/java/org/apache/accumulo/server/constraints/MetadataConstraints.java
+++
b/server/base/src/main/java/org/apache/accumulo/server/constraints/MetadataConstraints.java
@@ -130,9 +130,9 @@ public class MetadataConstraints implements Constraint {
}
/*
- * Validates the data file metadata is valid for a StoredDataFile.
+ * Validates the data file metadata is valid for a StoredTabletFile.
*/
- private static ArrayList<Short> validateDataFilePath(ArrayList<Short>
violations,
+ private static ArrayList<Short> validateDataFileMetadata(ArrayList<Short>
violations,
String metadata) {
try {
StoredTabletFile.validate(metadata);
@@ -216,8 +216,8 @@ public class MetadataConstraints implements Constraint {
}
if (columnFamily.equals(DataFileColumnFamily.NAME)) {
- violations =
- validateDataFilePath(violations, new
String(columnUpdate.getColumnQualifier(), UTF_8));
+ violations = validateDataFileMetadata(violations,
+ new String(columnUpdate.getColumnQualifier(), UTF_8));
try {
DataFileValue dfv = new DataFileValue(columnUpdate.getValue());
@@ -229,11 +229,11 @@ public class MetadataConstraints implements Constraint {
violations = addViolation(violations, 1);
}
} else if (columnFamily.equals(ScanFileColumnFamily.NAME)) {
- violations =
- validateDataFilePath(violations, new
String(columnUpdate.getColumnQualifier(), UTF_8));
+ violations = validateDataFileMetadata(violations,
+ new String(columnUpdate.getColumnQualifier(), UTF_8));
} else if (columnFamily.equals(BulkFileColumnFamily.NAME)) {
if (!columnUpdate.isDeleted() && !checkedBulk) {
- violations = validateDataFilePath(violations,
+ violations = validateDataFileMetadata(violations,
new String(columnUpdate.getColumnQualifier(), UTF_8));
// splits, which also write the time reference, are allowed to write
this reference even