[ https://issues.apache.org/jira/browse/HDFS-10480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16016815#comment-16016815 ]
Yiqun Lin commented on HDFS-10480: ---------------------------------- Thanks for the updated patch, [~manojg]! The latest patch almost looks good to me now. Only one comment from me: When the {{listOpenFiles}} command used in HA mode, the {{Tracer}} passed to {{OpenFilesIterator}} will be null. And this will lead a NPE error. The related codes in {{OpenFilesIterator}}. {code} + public BatchedEntries<OpenFileEntry> makeRequest(Long prevId) + throws IOException { + try (TraceScope ignored = tracer.newScope("listOpenFiles")) { <== there is a chance that tracer will be null + return namenode.listOpenFiles(prevId); + } + } {code} > Add an admin command to list currently open files > ------------------------------------------------- > > Key: HDFS-10480 > URL: https://issues.apache.org/jira/browse/HDFS-10480 > Project: Hadoop HDFS > Issue Type: Improvement > Reporter: Kihwal Lee > Assignee: Manoj Govindassamy > Attachments: HDFS-10480.02.patch, HDFS-10480.03.patch, > HDFS-10480-trunk-1.patch, HDFS-10480-trunk.patch > > > Currently there is no easy way to obtain the list of active leases or files > being written. It will be nice if we have an admin command to list open files > and their lease holders. -- This message was sent by Atlassian JIRA (v6.3.15#6346) --------------------------------------------------------------------- To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org