Skip to content

Conversation

@ruslanskorb
Copy link
Contributor

Motivation:

warning: 'wait()' is deprecated: Blocking API will be removed in favor of async await

Modifications:

Switches AskResponse.wait() over to await AskResponse.value

Result:

**Motivation:**

`warning: 'wait()' is deprecated: Blocking API will be removed in favor of async await`

**Modifications:**

Switches `AskResponse.wait()` over to `await AskResponse.value`

**Result:**

- Resolves apple#994
@swift-server-bot
Copy link

Can one of the admins verify this patch?

7 similar comments
@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@ktoso
Copy link
Member

ktoso commented Jul 11, 2022

@swift-server-bot test this please

@ktoso ktoso self-requested a review July 11, 2022 02:08
@ktoso
Copy link
Member

ktoso commented Jul 11, 2022

nightly failure was 11:15:27 Test Case 'ClusterLeaderActionsClusteredTests.test_singleLeader' failed (5.05 seconds) #945 known to be super flaky, I'll investigate it today, it keeps happening since NIO update.

@ktoso
Copy link
Member

ktoso commented Jul 11, 2022

Very nice, thank you @ruslanskorb !

@ktoso ktoso merged commit 63a59a7 into apple:main Jul 11, 2022
@ruslanskorb ruslanskorb deleted the iss994 branch July 11, 2022 05:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove all instances of blocking API use

3 participants