You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "Viraj Jasani (Jira)" <ji...@apache.org> on 2020/01/12 18:47:00 UTC
[jira] [Comment Edited] (HBASE-23569) Validate that the log cleaner
actually cleans oldWALs as expected
[ https://issues.apache.org/jira/browse/HBASE-23569?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17013846#comment-17013846 ]
Viraj Jasani edited comment on HBASE-23569 at 1/12/20 6:46 PM:
---------------------------------------------------------------
If we just check that after HMaster init, choreService has LogCleaner scheduled as part of the UT, should be good enough? Given that
# LogCleaner testing has it's own set of tests to ensure cleanup is fine
# UT might be preferable over IT since the frequency of UT run is quite high.
May be it’s good to have separate UT class where majority of default chores are ensured to have been scheduled and can be used going forward by any dev while introducing new chore(default/config based). As long as individual chore's functional testing is concerned, usually we would have separate UT class anyways.
Sounds good to you [~apurtell] ?
was (Author: vjasani):
If we just check that after HMaster init, choreService has LogCleaner scheduled as part of the UT, should be good enough? Given that LogCleaner testing has it's own set of tests to ensure cleanup is fine and UT might be preferable over IT since the frequency of UT run is quite high.
May be it’s good to have separate UT class where majority of default chores are ensured to have been scheduled and can be used going forward by any dev while introducing new chore(default/config based). As long as individual chore's functional testing is concerned, usually we would have separate UT class anyways.
Sounds good to you [~apurtell] ?
> Validate that the log cleaner actually cleans oldWALs as expected
> -----------------------------------------------------------------
>
> Key: HBASE-23569
> URL: https://issues.apache.org/jira/browse/HBASE-23569
> Project: HBase
> Issue Type: Test
> Components: integration tests, master, test
> Reporter: Andrew Kyle Purtell
> Priority: Major
> Fix For: 3.0.0, 2.3.0, 1.6.0
>
>
> The fix for HBASE-23287 (LogCleaner is not added to choreService) is in but we are lacking test coverage that validates that the log cleaner actually cleans oldWALs as expected. Add the test.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)