Apache9 commented on code in PR #6290:
URL: https://github.com/apache/hbase/pull/6290#discussion_r1837421966
##########
hbase-server/src/main/java/org/apache/hadoop/hbase/mob/MobUtils.java:
##########
@@ -309,7 +309,18 @@ public static void cleanExpiredMobFiles(FileSystem fs,
Configuration conf, Table
fileName = hfileLink.getOriginPath().getName();
}
- Date fileDate = parseDate(MobFileName.getDateFromName(fileName));
+ String s = String.format("%s=%s-",
Review Comment:
This should be a back reference file? Do we have some util method for
parsing this file name format? Or at least let's put all these methods
together? So in the future if there are some modifications we could know we
should change them all.
--
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.
To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
