Xiaoyu Yao commented on HDFS-13141:

[~chris.douglas] , thanks for the comments. Sorry, I just saw it after commit. 
{quote}bq. "Instead of extracting fields from one {{HdfsFileStatus}} to 
construct another, could {{SnapshottableDirectoryStatus}} just take the 
original struct, to avoid the {{convert}} method?"
Good point! I will work with [~ljain] with a new SnapshottableDirectoryStatus 
constructor that takes HdfsFileStatus directly to avoid the unnecessary 
construct and convert. 


public SnapshottableDirectoryStatus(HdfsFileStatus dirStatus,
 int snapshotNumber, int snapshotQuota, byte[] parentFullPath) {
  this.dirStatus = dirStatus;
  this.snapshotNumber = snapshotNumber;
  this.snapshotQuota = snapshotQuota;
  this.parentFullPath = parentFullPath;


> WebHDFS: Add support for getting snasphottable directory list
> -------------------------------------------------------------
>                 Key: HDFS-13141
>                 URL: https://issues.apache.org/jira/browse/HDFS-13141
>             Project: Hadoop HDFS
>          Issue Type: Task
>          Components: webhdfs
>            Reporter: Lokesh Jain
>            Assignee: Lokesh Jain
>            Priority: Major
>             Fix For: 3.1.0, 3.2.0
>         Attachments: HDFS-13141.001.patch, HDFS-13141.002.patch, 
> HDFS-13141.003.patch
> This Jira aims to implement get snapshottable directory list operation for 
> webHdfs filesystem.

This message was sent by Atlassian JIRA

To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to