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

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

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

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

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

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

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

2020-07-06 Thread GitBox
timmylicheng commented on a change in pull request #1049: URL: https://github.com/apache/hadoop-ozone/pull/1049#discussion_r450025900 ## File path: hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/pipeline/PipelineManagerV2Impl.java ## @@ -130,7 +134,7 @@

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

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

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

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

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

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

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

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