Skip to content

[Bug] Fix bug that checkpoint thread failed to start - #3795

Merged
morningman merged 1 commit into
apache:masterfrom
morningman:checkpoint_npe
Jun 8, 2020
Merged

[Bug] Fix bug that checkpoint thread failed to start#3795
morningman merged 1 commit into
apache:masterfrom
morningman:checkpoint_npe

Conversation

@morningman

Copy link
Copy Markdown
Contributor
  1. Set thread id before starting the checkpoint thread
  2. Init the CHECKPOINT catalog instance before visiting it.

Fix#3794

@morningmanmorningman added kind/fix Categorizes issue or PR as related to a bug. area/catalog Issues or PRs related to catalog management labels Jun 7, 2020
@morningmanmorningman self-assigned this Jun 7, 2020

@kangkaisenkangkaisen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, LGTM

@morningman

morningman commented Jun 8, 2020

Copy link
Copy Markdown
ContributorAuthor

Still has problem...
This PR has no problem.

I will create another PR to fix another problem

@morningman
morningman merged commit 8ada255 into apache:masterJun 8, 2020
yangzhg pushed a commit to yangzhg/doris that referenced this pull request Jun 10, 2020
1. Set thread id before starting the checkpoint thread
2. Init the CHECKPOINT catalog instance before visiting it.
morningman added a commit to morningman/doris that referenced this pull request Jun 22, 2020
1. Set thread id before starting the checkpoint thread
2. Init the CHECKPOINT catalog instance before visiting it.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/catalogIssues or PRs related to catalog managementkind/fixCategorizes issue or PR as related to a bug.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] FE checkpoint thread start failed with NPE

2 participants

@morningman@kangkaisen