[native] add blocking version of get_user_id method
ClosedPublic
Actions

Authored by varun on Aug 11 2022, 12:30 PM.

Details

Reviewers

tomek
• karol

Commits

rCOMM0130fbf2a304: [native] add blocking version of get_user_id method

Summary

Essentially converts the async get_user_id method to a sync method by blocking until the future is resolved. We do this because async functions are not allowed in the CXX bridge yet.

Depends on D4802

Test Plan

Successfully called the sync version and get a response from the Identity service.

Diff Detail

Repository

rCOMM Comm

Branch

identity-client (branched from master)

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

varun created this revision.Aug 11 2022, 12:30 PM

Herald added subscribers: • abosh, atul, • adrian, ashoat. · View Herald TranscriptAug 11 2022, 12:30 PM

Harbormaster returned this revision to the author for changes because remote builds failed.Aug 11 2022, 12:31 PM

Harbormaster failed remote builds in B11322: Diff 15579!

Android build failure is unrelated

varun added a child revision: D4820: [native] set up CXX bridge.Aug 11 2022, 8:43 PM

tomek accepted this revision.Aug 12 2022, 3:55 AM

This revision is now accepted and ready to land.Aug 12 2022, 3:55 AM

• karol accepted this revision.Aug 12 2022, 4:51 AM

This revision was landed with ongoing or failed builds.Aug 18 2022, 11:34 AM

Closed by commit rCOMM0130fbf2a304: [native] add blocking version of get_user_id method (authored by varun). · Explain Why

This revision was automatically updated to reflect the committed changes.

varun added a commit: rCOMM0130fbf2a304: [native] add blocking version of get_user_id method.

Revision Contents
Changeset List

Path

Size

native/

cpp/

CommonCpp/

grpc/

grpc_client/

src/

lib.rs

10 lines

Diff 15579

View Options

native/cpp/CommonCpp/grpc/grpc_client/src/lib.rs

[native] add blocking version of get_user_id methodClosedPublicActions