busbey commented on a change in pull request #921: HBASE-22749: Distributed MOB compactions URL: https://github.com/apache/hbase/pull/921#discussion_r376081350
########## File path: hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFileInfo.java ########## @@ -443,22 +430,42 @@ public static boolean isHFile(final String fileName) { } /** - * @param path Path to check. - * @return True if the path has format of a del file. + * Checks if the file is a MOB file + * @param path path to a file + * @return true, if - yes, false otherwise */ - public static boolean isDelFile(final Path path) { - return isDelFile(path.getName()); + public static boolean isMobFile(final Path path) { Review comment: Just to make sure I understand, we're purposefully going to return `false` for any files in the MOB storage area that are from prior to the distributed mob compaction changes? ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services