Re: [PR] YARN-9013. [BackPort] [GPG] fix order of steps cleaning Registry entries in ApplicationCleaner. [hadoop]

2023-10-30 Thread via GitHub
slfan1989 commented on PR #6147: URL: https://github.com/apache/hadoop/pull/6147#issuecomment-1786173168 @goiri Thank you very much for your help in reviewing the code! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [PR] YARN-9013. [BackPort] [GPG] fix order of steps cleaning Registry entries in ApplicationCleaner. [hadoop]

2023-10-30 Thread via GitHub
slfan1989 merged PR #6147: URL: https://github.com/apache/hadoop/pull/6147 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] YARN-9013. [BackPort] [GPG] fix order of steps cleaning Registry entries in ApplicationCleaner. [hadoop]

2023-10-26 Thread via GitHub
hadoop-yetus commented on PR #6147: URL: https://github.com/apache/hadoop/pull/6147#issuecomment-1782253404 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

Re: [PR] YARN-9013. [BackPort] [GPG] fix order of steps cleaning Registry entries in ApplicationCleaner. [hadoop]

2023-10-21 Thread via GitHub
hadoop-yetus commented on PR #6147: URL: https://github.com/apache/hadoop/pull/6147#issuecomment-1773868203 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

Re: [PR] YARN-9013. [BackPort] [GPG] fix order of steps cleaning Registry entries in ApplicationCleaner. [hadoop]

2023-10-19 Thread via GitHub
hadoop-yetus commented on PR #6147: URL: https://github.com/apache/hadoop/pull/6147#issuecomment-1771671879 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

Re: [PR] YARN-9013. [BackPort] [GPG] fix order of steps cleaning Registry entries in ApplicationCleaner. [hadoop]

2023-10-19 Thread via GitHub
slfan1989 commented on code in PR #6147: URL: https://github.com/apache/hadoop/pull/6147#discussion_r1365878423 ##

Re: [PR] YARN-9013. [BackPort] [GPG] fix order of steps cleaning Registry entries in ApplicationCleaner. [hadoop]

2023-10-19 Thread via GitHub
slfan1989 commented on code in PR #6147: URL: https://github.com/apache/hadoop/pull/6147#discussion_r1365877007 ## hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/conf/YarnConfiguration.java: ## @@ -4367,7 +4367,7 @@ public static boolean

Re: [PR] YARN-9013. [BackPort] [GPG] fix order of steps cleaning Registry entries in ApplicationCleaner. [hadoop]

2023-10-17 Thread via GitHub
goiri commented on code in PR #6147: URL: https://github.com/apache/hadoop/pull/6147#discussion_r1362534634 ## hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/conf/YarnConfiguration.java: ## @@ -4367,7 +4367,7 @@ public static boolean

Re: [PR] YARN-9013. [BackPort] [GPG] fix order of steps cleaning Registry entries in ApplicationCleaner. [hadoop]

2023-10-16 Thread via GitHub
hadoop-yetus commented on PR #6147: URL: https://github.com/apache/hadoop/pull/6147#issuecomment-1765680325 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

Re: [PR] YARN-9013. [BackPort] [GPG] fix order of steps cleaning Registry entries in ApplicationCleaner. [hadoop]

2023-10-14 Thread via GitHub
hadoop-yetus commented on PR #6147: URL: https://github.com/apache/hadoop/pull/6147#issuecomment-1763250012 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

Re: [PR] YARN-9013. [BackPort] [GPG] fix order of steps cleaning Registry entries in ApplicationCleaner. [hadoop]

2023-10-12 Thread via GitHub
hadoop-yetus commented on PR #6147: URL: https://github.com/apache/hadoop/pull/6147#issuecomment-1760867408 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

Re: [PR] YARN-9013. [BackPort] [GPG] fix order of steps cleaning Registry entries in ApplicationCleaner. [hadoop]

2023-10-11 Thread via GitHub
goiri commented on code in PR #6147: URL: https://github.com/apache/hadoop/pull/6147#discussion_r1355745625 ##

Re: [PR] YARN-9013. [BackPort] [GPG] fix order of steps cleaning Registry entries in ApplicationCleaner. [hadoop]

2023-10-10 Thread via GitHub
hadoop-yetus commented on PR #6147: URL: https://github.com/apache/hadoop/pull/6147#issuecomment-1755859863 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

Re: [PR] YARN-9013. [BackPort] [GPG] fix order of steps cleaning Registry entries in ApplicationCleaner. [hadoop]

2023-10-04 Thread via GitHub
hadoop-yetus commented on PR #6147: URL: https://github.com/apache/hadoop/pull/6147#issuecomment-1746636995 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: |

[PR] YARN-9013. [BackPort] [GPG] fix order of steps cleaning Registry entries in ApplicationCleaner. [hadoop]

2023-10-04 Thread via GitHub
slfan1989 opened a new pull request, #6147: URL: https://github.com/apache/hadoop/pull/6147 ### Description of PR JIRA: YARN-9013. [BackPort] [GPG] fix order of steps cleaning Registry entries in ApplicationCleaner. ### How was this patch tested? Add Junit Test.