[jira] [Commented] (HDFS-15520) Use visitor pattern to visit namespace tree

2020-08-11 Thread Tsz-wo Sze (Jira)


[ 
https://issues.apache.org/jira/browse/HDFS-15520?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17175875#comment-17175875
 ] 

Tsz-wo Sze commented on HDFS-15520:
---

Thanks for pointing out. Will fix it.

> Use visitor pattern to visit namespace tree
> ---
>
> Key: HDFS-15520
> URL: https://issues.apache.org/jira/browse/HDFS-15520
> Project: Hadoop HDFS
>  Issue Type: Improvement
>  Components: namenode
>Reporter: Tsz-wo Sze
>Assignee: Tsz-wo Sze
>Priority: Major
> Fix For: 3.4.0
>
>
> In order to allow the FsImageValidation tool to verify the namespace 
> structure, we use a visitor pattern so that the tool can visit all the INodes 
> and all the snapshots in the namespace tree.
> The existing INode.dumpTreeRecursively() can also be implemented by a visitor.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

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



[jira] [Commented] (HDFS-15520) Use visitor pattern to visit namespace tree

2020-08-11 Thread Stephen O'Donnell (Jira)


[ 
https://issues.apache.org/jira/browse/HDFS-15520?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17175871#comment-17175871
 ] 

Stephen O'Donnell commented on HDFS-15520:
--

[~szetszwo] This change seems to have introduced a two new find bugs warnings:

https://github.com/apache/hadoop/pull/2203#issuecomment-671841846

Could you check and resolve them please?

> Use visitor pattern to visit namespace tree
> ---
>
> Key: HDFS-15520
> URL: https://issues.apache.org/jira/browse/HDFS-15520
> Project: Hadoop HDFS
>  Issue Type: Improvement
>  Components: namenode
>Reporter: Tsz-wo Sze
>Assignee: Tsz-wo Sze
>Priority: Major
> Fix For: 3.4.0
>
>
> In order to allow the FsImageValidation tool to verify the namespace 
> structure, we use a visitor pattern so that the tool can visit all the INodes 
> and all the snapshots in the namespace tree.
> The existing INode.dumpTreeRecursively() can also be implemented by a visitor.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

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



[jira] [Commented] (HDFS-15520) Use visitor pattern to visit namespace tree

2020-08-11 Thread Mingliang Liu (Jira)


[ 
https://issues.apache.org/jira/browse/HDFS-15520?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17175685#comment-17175685
 ] 

Mingliang Liu commented on HDFS-15520:
--

Let's link PRs to JIRAs manually if they are not showing automatically?

> Use visitor pattern to visit namespace tree
> ---
>
> Key: HDFS-15520
> URL: https://issues.apache.org/jira/browse/HDFS-15520
> Project: Hadoop HDFS
>  Issue Type: Improvement
>  Components: namenode
>Reporter: Tsz-wo Sze
>Assignee: Tsz-wo Sze
>Priority: Major
> Fix For: 3.4.0
>
>
> In order to allow the FsImageValidation tool to verify the namespace 
> structure, we use a visitor pattern so that the tool can visit all the INodes 
> and all the snapshots in the namespace tree.
> The existing INode.dumpTreeRecursively() can also be implemented by a visitor.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

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