Re: [PR] HBASE-24687 MobFileCleanerChore uses a new Connection for each table … [hbase]

2023-10-26 Thread via GitHub
bbeaudreault commented on code in PR #2038: URL: https://github.com/apache/hbase/pull/2038#discussion_r1373746594 ## hbase-server/src/main/java/org/apache/hadoop/hbase/mob/MobFileCleanerChore.java: ## @@ -103,38 +104,38 @@ private void checkObsoleteConfigurations() { @Visib

Re: [PR] HBASE-24687 MobFileCleanerChore uses a new Connection for each table … [hbase]

2023-10-26 Thread via GitHub
bbeaudreault commented on code in PR #2038: URL: https://github.com/apache/hbase/pull/2038#discussion_r1373744967 ## hbase-server/src/main/java/org/apache/hadoop/hbase/mob/MobFileCleanerChore.java: ## @@ -156,140 +156,143 @@ public void cleanupObsoleteMobFiles(Configuration conf

Re: [PR] HBASE-24687 MobFileCleanerChore uses a new Connection for each table … [hbase]

2023-10-26 Thread via GitHub
bbeaudreault commented on code in PR #2038: URL: https://github.com/apache/hbase/pull/2038#discussion_r1373743878 ## hbase-server/src/main/java/org/apache/hadoop/hbase/mob/MobFileCleanerChore.java: ## @@ -103,38 +104,38 @@ private void checkObsoleteConfigurations() { @Visib

Re: [PR] HBASE-24687 MobFileCleanerChore uses a new Connection for each table … [hbase]

2023-10-26 Thread via GitHub
bbeaudreault commented on PR #2038: URL: https://github.com/apache/hbase/pull/2038#issuecomment-1781800783 This PR has become long out of date, with a bunch of merge conflicts. Closing it and we will re-implement it based on the current state of the branches. -- This is an automated mess

Re: [PR] HBASE-24687 MobFileCleanerChore uses a new Connection for each table … [hbase]

2023-10-26 Thread via GitHub
bbeaudreault closed pull request #2038: HBASE-24687 MobFileCleanerChore uses a new Connection for each table … URL: https://github.com/apache/hbase/pull/2038 -- 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 t

Re: [PR] HBASE-24687 MobFileCleanerChore uses a new Connection for each table … [hbase]

2023-10-16 Thread via GitHub
bbeaudreault commented on PR #2038: URL: https://github.com/apache/hbase/pull/2038#issuecomment-1764825284 @ArthurSXL8 any chance you can rebase and fix the merge conflicts on this? I can help get it merged, this recently became an issue for us. -- This is an automated message from the Ap