Skip to content

Broker: Start up DruidSchema immediately if there are no segments.#6765

Merged
dclim merged 1 commit intoapache:masterfrom
gianm:fix-delayed-startup
Dec 20, 2018
Merged

Broker: Start up DruidSchema immediately if there are no segments.#6765
dclim merged 1 commit intoapache:masterfrom
gianm:fix-delayed-startup

Conversation

@gianm
Copy link
Copy Markdown
Contributor

@gianm gianm commented Dec 20, 2018

Fixes a bug introduced in #6742, where the broker would delay startup
indefinitely if there were no segments at all being served by any
data servers.

Fixes a bug introduced in apache#6742, where the broker would delay startup
indefinitely if there were no segments at all being served by any
data servers.
@gianm gianm added the Bug label Dec 20, 2018
@gianm gianm added this to the 0.14.0 milestone Dec 20, 2018
@dclim
Copy link
Copy Markdown
Contributor

dclim commented Dec 20, 2018

👍

@dclim dclim merged commit f0b7c27 into apache:master Dec 20, 2018
@gianm gianm deleted the fix-delayed-startup branch December 20, 2018 18:09
dclim pushed a commit to implydata/druid-public that referenced this pull request Dec 20, 2018
…pache#6765)

Fixes a bug introduced in apache#6742, where the broker would delay startup
indefinitely if there were no segments at all being served by any
data servers.
@zhangyue19921010
Copy link
Copy Markdown
Contributor

zhangyue19921010 commented Jan 20, 2021

Nice feature for Await initialization before finishing startup! But I met a bug introduced in this PR which will not wait for broker SQL metadata view to fully initialize before starting up, even though set awaitInitializationOnStart true. I just make a PR #10779 trying to fix it :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants