You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Ádám Bakai (Code Review)" <ge...@cloudera.org> on 2023/04/03 14:15:00 UTC

[kudu-CR] Change order of operations during InitCatalogManagerTask

Hello Kudu Jenkins, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/19639

to look at the new patch set (#2).

Change subject: Change order of operations during InitCatalogManagerTask
......................................................................

Change order of operations during InitCatalogManagerTask

In MasterStartupTest. StartupWebPage test, WaitForCatalogManagerInit is
executed, which waits until catalog_manager_init_status_ is set. It is
set by InitCatalogManagerTask and with the previous order of operations
it was possible (but very rare) that catalog_manager_init_status_ was
set but the timer was not stopped at the time of the startup http
request, which resulted an output where
initialize_master_catalog_status was 0. With this order
initialize_master_catalog_progress_ is always set if
WaitForCatalogManagerInit is finished.

Change-Id: I64c980a8b357c99d4431a02d58b3c03a60bae1ad
---
M src/kudu/master/master.cc
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/39/19639/2
-- 
To view, visit http://gerrit.cloudera.org:8080/19639
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I64c980a8b357c99d4431a02d58b3c03a60bae1ad
Gerrit-Change-Number: 19639
Gerrit-PatchSet: 2
Gerrit-Owner: Ádám Bakai <ab...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)