[GitHub] [hadoop-ozone] xiaoyuyao commented on a change in pull request #1049: HDDS-3662 Decouple finalizeAndDestroyPipeline.

2020-07-07 Thread GitBox
xiaoyuyao commented on a change in pull request #1049: URL: https://github.com/apache/hadoop-ozone/pull/1049#discussion_r451023617 ## File path: hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/pipeline/PipelineManagerV2Impl.java ## @@ -410,18 +399,29 @@ public

[GitHub] [hadoop-ozone] xiaoyuyao commented on a change in pull request #1049: HDDS-3662 Decouple finalizeAndDestroyPipeline.

2020-07-07 Thread GitBox
xiaoyuyao commented on a change in pull request #1049: URL: https://github.com/apache/hadoop-ozone/pull/1049#discussion_r451024886 ## File path: hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/pipeline/SCMPipelineManager.java ## @@ -421,18 +438,29 @@ public

[GitHub] [hadoop-ozone] xiaoyuyao commented on a change in pull request #1049: HDDS-3662 Decouple finalizeAndDestroyPipeline.

2020-07-07 Thread GitBox
xiaoyuyao commented on a change in pull request #1049: URL: https://github.com/apache/hadoop-ozone/pull/1049#discussion_r451024282 ## File path: hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/pipeline/PipelineManagerV2Impl.java ## @@ -310,94 +321,72 @@ public

[GitHub] [hadoop-ozone] xiaoyuyao commented on a change in pull request #1049: HDDS-3662 Decouple finalizeAndDestroyPipeline.

2020-07-07 Thread GitBox
xiaoyuyao commented on a change in pull request #1049: URL: https://github.com/apache/hadoop-ozone/pull/1049#discussion_r451023617 ## File path: hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/pipeline/PipelineManagerV2Impl.java ## @@ -410,18 +399,29 @@ public

[GitHub] [hadoop-ozone] xiaoyuyao commented on a change in pull request #1049: HDDS-3662 Decouple finalizeAndDestroyPipeline.

2020-06-11 Thread GitBox
xiaoyuyao commented on a change in pull request #1049: URL: https://github.com/apache/hadoop-ozone/pull/1049#discussion_r439130260 ## File path: hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/pipeline/PipelineManagerV2Impl.java ## @@ -310,102 +315,69 @@

[GitHub] [hadoop-ozone] xiaoyuyao commented on a change in pull request #1049: HDDS-3662 Decouple finalizeAndDestroyPipeline.

2020-06-11 Thread GitBox
xiaoyuyao commented on a change in pull request #1049: URL: https://github.com/apache/hadoop-ozone/pull/1049#discussion_r439132746 ## File path: hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/pipeline/PipelineManagerV2Impl.java ## @@ -420,10 +392,50 @@ public

[GitHub] [hadoop-ozone] xiaoyuyao commented on a change in pull request #1049: HDDS-3662 Decouple finalizeAndDestroyPipeline.

2020-06-11 Thread GitBox
xiaoyuyao commented on a change in pull request #1049: URL: https://github.com/apache/hadoop-ozone/pull/1049#discussion_r439132593 ## File path: hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/pipeline/PipelineManagerV2Impl.java ## @@ -310,102 +315,69 @@

[GitHub] [hadoop-ozone] xiaoyuyao commented on a change in pull request #1049: HDDS-3662 Decouple finalizeAndDestroyPipeline.

2020-06-11 Thread GitBox
xiaoyuyao commented on a change in pull request #1049: URL: https://github.com/apache/hadoop-ozone/pull/1049#discussion_r439132303 ## File path: hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/pipeline/PipelineManagerV2Impl.java ## @@ -420,10 +392,50 @@ public

[GitHub] [hadoop-ozone] xiaoyuyao commented on a change in pull request #1049: HDDS-3662 Decouple finalizeAndDestroyPipeline.

2020-06-11 Thread GitBox
xiaoyuyao commented on a change in pull request #1049: URL: https://github.com/apache/hadoop-ozone/pull/1049#discussion_r439130260 ## File path: hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/pipeline/PipelineManagerV2Impl.java ## @@ -310,102 +315,69 @@