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

[PyTorch Docs] Updated RRef docs to indicate RPC Retries #36678

Closed
wants to merge 1 commit into from

Conversation

osalpekar
Copy link
Member

@osalpekar osalpekar commented Apr 15, 2020

Stack from ghstack:

Updated the docs to explicitly indicate that RRef control messages are
idempotent and retried upon failure.

Differential Revision: D20828041

Updated the docs to explicitly indicate that RRef control messages are
idempotent and retried upon failure.

Differential Revision: [D20828041](https://our.internmc.facebook.com/intern/diff/D20828041/)

[ghstack-poisoned]
osalpekar added a commit that referenced this pull request Apr 15, 2020
Updated the docs to explicitly indicate that RRef control messages are
idempotent and retried upon failure.

Differential Revision: [D20828041](https://our.internmc.facebook.com/intern/diff/D20828041/)

ghstack-source-id: 102225791
Pull Request resolved: #36678
@dr-ci
Copy link

dr-ci bot commented Apr 16, 2020

💊 Build failures summary and remediations

As of commit e454a1c (more details on the Dr. CI page):


💚 💚 Looks good so far! There are no failures yet. 💚 💚


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions on the GitHub issue tracker.

See how this bot performed.

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 5927a67.

@facebook-github-bot facebook-github-bot deleted the gh/osalpekar/12/head branch April 19, 2020 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants