yihua commented on code in PR #13873:
URL: https://github.com/apache/hudi/pull/13873#discussion_r2334125296
##########
hudi-io/src/main/java/org/apache/hudi/io/hfile/HFileWriterImpl.java:
##########
@@ -219,6 +219,41 @@ private void initFileInfo() {
new byte[]{0});
}
+ protected void finishFileInfo() {
Review Comment:
Let's add a functional test that can reproduce the issue, at the HFile
reader and writer level i.e., the test fails before the fix and passes after
the fix. We can add HBase as test dependency for such a test on compatibility.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
