Skip to content

Commit

Permalink
[#16408] YSQL: Split TestPgRegressIndex.testPgRegressIndex
Browse files Browse the repository at this point in the history
Summary:
The unit test org.yb.pgsql.TestPgRegressIndex.testPgRegressIndex oftens times
out with an error like:
```
org.junit.runners.model.TestTimedOutException: test timed out after 1800 seconds
```

I splitted the schedule `yb_index_serial_schedule` into two so that each has a
better chance not to time out. They each take about the same run time.
Jira: DB-5819

Test Plan:
./yb_build.sh --java-test 'org.yb.pgsql.TestPgRegressIndex#testPgRegressIndex'
./yb_build.sh --java-test 'org.yb.pgsql.TestPgRegressIndex#testPgRegressIndex2'

On my dev vm debug build, testPgRegressIndex runs about 18:30 minutes,
testPgRegressIndex2 runs about 19 minutes.

Reviewers: fizaa

Reviewed By: fizaa

Subscribers: yql

Differential Revision: https://phorge.dev.yugabyte.com/D36224
  • Loading branch information
myang2021 committed Jun 28, 2024
1 parent b239e07 commit ec76062
Show file tree
Hide file tree
Showing 3 changed files with 20 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -49,4 +49,9 @@ protected Map<String, String> getTServerFlags() {
public void testPgRegressIndex() throws Exception {
runPgRegressTest("yb_index_serial_schedule");
}

@Test
public void testPgRegressIndex2() throws Exception {
runPgRegressTest("yb_index_serial2_schedule");
}
}
15 changes: 15 additions & 0 deletions src/postgres/src/test/regress/yb_index_serial2_schedule
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
# src/test/regress/yb_index_serial_schedule
#
####################################################################################################
# This testsuite includes miscellaneous tests that are not yet characterized to a proper testsuite.
# The tests in this schedule are not ported from PostgreSQL original tests except for those that
# are prefixed with "yb_pg".
####################################################################################################
test: yb_create_index
test: yb_index_scan
test: yb_reindex
test: yb_secondary_index_scan

test: yb_index_scan_null_create
test: yb_index_scan_null_hash
test: yb_index_scan_null_asc
9 changes: 0 additions & 9 deletions src/postgres/src/test/regress/yb_index_serial_schedule
Original file line number Diff line number Diff line change
Expand Up @@ -7,12 +7,3 @@
####################################################################################################
test: yb_pg_indexing
test: yb_indexing_order

test: yb_create_index
test: yb_index_scan
test: yb_reindex
test: yb_secondary_index_scan

test: yb_index_scan_null_create
test: yb_index_scan_null_hash
test: yb_index_scan_null_asc

0 comments on commit ec76062

Please sign in to comment.