You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by la...@apache.org on 2023/11/06 05:02:35 UTC

(kudu) 01/02: Fix the missing parameterized unit test

This is an automated email from the ASF dual-hosted git repository.

laiyingchun pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git

commit 0ce3e14f4d9d2f70cfa87952fb83df1e2f619945
Author: Yingchun Lai <la...@apache.org>
AuthorDate: Sun Nov 5 23:33:56 2023 +0800

    Fix the missing parameterized unit test
    
    Change-Id: Id06947846a401183d00e00bf43d7cf4d86002232
    Reviewed-on: http://gerrit.cloudera.org:8080/20656
    Tested-by: Yingchun Lai <la...@apache.org>
    Reviewed-by: Yifan Zhang <ch...@163.com>
---
 src/kudu/integration-tests/ts_recovery-itest.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/kudu/integration-tests/ts_recovery-itest.cc b/src/kudu/integration-tests/ts_recovery-itest.cc
index 31d630c3f..df7eaef90 100644
--- a/src/kudu/integration-tests/ts_recovery-itest.cc
+++ b/src/kudu/integration-tests/ts_recovery-itest.cc
@@ -146,7 +146,7 @@ void TsRecoveryITest::StartClusterOneTs(vector<string> extra_tserver_flags,
 // being unable to complete replay of the WAL and leaves the WAL recovery directory in place.
 // 3. The master should tombstone the FAILED replica, causing its recovery directory to be deleted.
 // A subsequent tablet copy and tablet bootstrap should cause the replica to become healthy again.
-TEST_F(TsRecoveryITest, TestTabletRecoveryAfterSegmentDelete) {
+TEST_P(TsRecoveryITest, TestTabletRecoveryAfterSegmentDelete) {
   // Start a cluster with 3 tablet servers consisting of 1 tablet with 3 replicas.
   // Configure a small log segment size to quickly create new log segments.
   // Since we want to write as quickly as possible, we disable WAL compression.