Apache9 commented on code in PR #5545: URL: https://github.com/apache/hbase/pull/5545#discussion_r1558971974
########## hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFileWriter.java: ########## @@ -256,156 +234,571 @@ public void appendMobMetadata(SetMultimap<TableName, String> mobRefSet) throws I public void appendTrackedTimestampsToMetadata() throws IOException { // TODO: The StoreFileReader always converts the byte[] to TimeRange // via TimeRangeTracker, so we should write the serialization data of TimeRange directly. - appendFileInfo(TIMERANGE_KEY, TimeRangeTracker.toByteArray(timeRangeTracker)); - appendFileInfo(EARLIEST_PUT_TS, Bytes.toBytes(earliestPutTs)); + liveFileWriter.appendTrackedTimestampsToMetadata(); Review Comment: Checked the code, besides calling the (old) StoreFileWriter, this method is only called in HFileOutputFormat2, for appending metadata for store files generated by MR job. Seems better to introduce a special method in StoreFileWriter for it. Anyway, not a problem for this PR. Can file another issue for it. -- 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