[GitHub] [hbase] apurtell commented on pull request #3230: HBASE-25847 More DEBUG and TRACE level logging in CatalogJanitor and HbckChore

2021-05-05 Thread GitBox
apurtell commented on pull request #3230: URL: https://github.com/apache/hbase/pull/3230#issuecomment-833124523 Added log level guards as requested before merge. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [hbase] apurtell commented on pull request #3230: HBASE-25847 More DEBUG and TRACE level logging in CatalogJanitor and HbckChore

2021-05-05 Thread GitBox
apurtell commented on pull request #3230: URL: https://github.com/apache/hbase/pull/3230#issuecomment-832853177 Just the logging changes now, and the bug fix to ensure log lines emit correct information. The CatalogJanitor cleanup proposal is now this PR:

[GitHub] [hbase] apurtell commented on pull request #3230: HBASE-25847 More DEBUG and TRACE level logging in CatalogJanitor and HbckChore

2021-05-05 Thread GitBox
apurtell commented on pull request #3230: URL: https://github.com/apache/hbase/pull/3230#issuecomment-832843132 Well since what I want here is "More DEBUG and TRACE level logging" I will remove the additional changes so the test failure is fixed and address that on a separate PR. --

[GitHub] [hbase] apurtell commented on pull request #3230: HBASE-25847 More DEBUG and TRACE level logging in CatalogJanitor and HbckChore

2021-05-05 Thread GitBox
apurtell commented on pull request #3230: URL: https://github.com/apache/hbase/pull/3230#issuecomment-832841115 With these changes the ingestion test completes successfully and without weird artifacts in the logging. All split regions are GCed by procedures. In memory state aligns with