ThinkerLei commented on code in PR #6384:
URL: https://github.com/apache/hudi/pull/6384#discussion_r964412660
##########
hudi-common/src/main/java/org/apache/hudi/common/model/HoodieLogFile.java:
##########
@@ -98,7 +111,10 @@ public String getFileExtension() {
}
public Path getPath() {
- return new Path(pathStr);
+ if (path == null) {
+ this.path = new Path(pathStr);
Review Comment:
@yihua Thanks for your comments and suggestions，I have taken your suggestion
and submitted again. I haven't compared the memory usage before and after the
patch in time. But with four new variables we can roughly estimate the memory
usage.
--
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]
