[accumulo] branch main updated: Minor fix for naming of method/comment in MetadataConstraints

[cs...@apache.org]

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) <ch...@gmail.com>
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