Skip to content

fix(core): Don't wake up operator futures while not ready#2415

Merged
Xuanwo merged 1 commit intomainfrom
fix-future
Jun 5, 2023
Merged

fix(core): Don't wake up operator futures while not ready#2415
Xuanwo merged 1 commit intomainfrom
fix-future

Conversation

@Xuanwo
Copy link
Copy Markdown
Member

@Xuanwo Xuanwo commented Jun 5, 2023

Fix #2412

Signed-off-by: Xuanwo <github@xuanwo.io>
@Xuanwo Xuanwo marked this pull request as ready for review June 5, 2023 07:54
@github-actions github-actions Bot added the releases-note/fix The PR fixes a bug or has a title that begins with "fix" label Jun 5, 2023
@Xuanwo Xuanwo requested a review from suyanhanx June 5, 2023 07:57
Copy link
Copy Markdown
Contributor

@ClSlaid ClSlaid left a comment

Choose a reason for hiding this comment

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

I think a series of blog posts are needed to explain how it broke and fixed

@Xuanwo Xuanwo merged commit fe21ea7 into main Jun 5, 2023
@Xuanwo Xuanwo deleted the fix-future branch June 5, 2023 08:07
Copy link
Copy Markdown
Member

@suyanhanx suyanhanx left a comment

Choose a reason for hiding this comment

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

Thank you for your hard work.

@suyanhanx suyanhanx mentioned this pull request Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/fix The PR fixes a bug or has a title that begins with "fix"

Projects

None yet

Development

Successfully merging this pull request may close these issues.

OpenDAL v0.36.0 on windows has a strange 60s hang

4 participants