voonhous commented on code in PR #17830:
URL: https://github.com/apache/hudi/pull/17830#discussion_r2692963894


##########
hudi-common/src/main/java/org/apache/hudi/common/model/BaseFile.java:
##########
@@ -55,66 +69,13 @@ public BaseFile(String filePath) {
     this(null, filePath, getFileName(filePath), -1);
   }
 
-  private BaseFile(StoragePathInfo pathInfo, String fullPath, String fileName, 
long fileLen) {
-    this.pathInfo = pathInfo;
-    this.fullPath = fullPath;
-    this.fileLen = fileLen;
-    this.fileName = fileName;
-  }
-
-  public String getPath() {

Review Comment:
   Done, I've change the attribute/field to `path` from `fullPath` so that the 
getter generated by Lombok is `getPath`. 
   
   On top of that, i've added a comment. This will reduce scope of the 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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to