Skip to content

Commit

Permalink
[#25404] DocDB: Fix MasterTest.TestListTablesIncludesIndexedTableId i…
Browse files Browse the repository at this point in the history
…n TSAN

Summary:
Test uses CreateNamespaceAsync, but does not wait for completion.
As result it tries to create table in namespace that is not yet ready.

Fixed by using CreateNamespace.
Jira: DB-14635

Test Plan: ./yb_build.sh --gtest_filter MasterTest.TestListTablesIncludesIndexedTableId -n 40

Reviewers: hsunder, slingam

Reviewed By: hsunder

Subscribers: ybase

Tags: #jenkins-ready

Differential Revision: https://phorge.dev.yugabyte.com/D40843
  • Loading branch information
spolitov committed Dec 22, 2024
1 parent 124135a commit 79c1abb
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/yb/master/master-test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -941,7 +941,7 @@ TEST_F(MasterTest, TestListTablesIncludesIndexedTableId) {
NamespaceName test_name = "test_pgsql";
CreateNamespaceResponsePB resp;
NamespaceId nsid;
ASSERT_OK(CreateNamespaceAsync(test_name, YQLDatabase::YQL_DATABASE_PGSQL, &resp));
ASSERT_OK(CreateNamespace(test_name, YQLDatabase::YQL_DATABASE_PGSQL, &resp));
nsid = resp.id();

const Schema kTableSchema({
Expand Down

0 comments on commit 79c1abb

Please sign in to comment.