Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[promises] Make Poll<T> its own type, not a variant<> #32515

Merged
merged 14 commits into from
Mar 3, 2023

Conversation

ctiller
Copy link
Member

@ctiller ctiller commented Mar 2, 2023

Avoids some compilation problems on older MSVC's, opens the door for some future optimizations.

@markdroth markdroth changed the title [promises] Make Poll<T> its own type, not a variant<> [master version] [promises] Make Poll<T> its own type, not a variant<> Mar 2, 2023
@ctiller ctiller enabled auto-merge (squash) March 2, 2023 22:41
@ctiller ctiller merged commit 0d7b344 into grpc:master Mar 3, 2023
@ctiller ctiller deleted the fix-gcs-m branch March 3, 2023 01:44
@copybara-service copybara-service bot added the imported Specifies if the PR has been imported to the internal repository label Mar 3, 2023
markdroth pushed a commit to markdroth/grpc that referenced this pull request Mar 3, 2023
Avoids some compilation problems on older MSVC's, opens the door for
some future optimizations.

---------

Co-authored-by: ctiller <ctiller@users.noreply.github.com>
markdroth added a commit that referenced this pull request Mar 3, 2023
drfloob added a commit that referenced this pull request Mar 3, 2023
It seems as if we ran into this bug
actions/labeler#442 on at least one PR
#32515. This PR utilizes the suggested
workaround until we can upgrade to a new major version release of
actions/labeler.
XuanWang-Amos pushed a commit to XuanWang-Amos/grpc that referenced this pull request May 1, 2023
Avoids some compilation problems on older MSVC's, opens the door for
some future optimizations.

---------

Co-authored-by: ctiller <ctiller@users.noreply.github.com>
XuanWang-Amos pushed a commit to XuanWang-Amos/grpc that referenced this pull request May 1, 2023
…32542)

It seems as if we ran into this bug
actions/labeler#442 on at least one PR
grpc#32515. This PR utilizes the suggested
workaround until we can upgrade to a new major version release of
actions/labeler.
wanlin31 pushed a commit that referenced this pull request May 18, 2023
Avoids some compilation problems on older MSVC's, opens the door for
some future optimizations.

---------

Co-authored-by: ctiller <ctiller@users.noreply.github.com>
wanlin31 pushed a commit that referenced this pull request May 18, 2023
It seems as if we ran into this bug
actions/labeler#442 on at least one PR
#32515. This PR utilizes the suggested
workaround until we can upgrade to a new major version release of
actions/labeler.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bloat/improvement imported Specifies if the PR has been imported to the internal repository lang/core per-call-memory/neutral per-channel-memory/neutral release notes: no Indicates if PR should not be in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants