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

Rename I{Numerical, String, List, Map}Column #226

Closed
wants to merge 1 commit into from

Conversation

wenleix
Copy link
Contributor

@wenleix wenleix commented Mar 2, 2022

Differential Revision: D34572815

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported labels Mar 2, 2022
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D34572815

wenleix added a commit to wenleix/torcharrow that referenced this pull request Mar 2, 2022
Summary: Pull Request resolved: pytorch#226

Differential Revision: D34572815

fbshipit-source-id: b82f48b187848af53bbc81f1d74f9c8feff56111
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D34572815

Summary: Pull Request resolved: pytorch#226

Reviewed By: Tianshu-Bao

Differential Revision: D34572815

fbshipit-source-id: 013bbd267b58608cfd76fc0b5bc76c9c3208d38e
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D34572815

YLGH pushed a commit to YLGH/torcharrow that referenced this pull request May 7, 2022
Summary:
Pull Request resolved: pytorch/torchrec#226

setException and thenTry will cause the exception to be thrown and catch to handle the error logic, this makes the exception handling path extremely expensive, ideally we want to make error path less expensive than normal path to make sure we can recover from an overload situation

Reviewed By: yinghai

Differential Revision: D35669573

fbshipit-source-id: 69ff294f1c1ec37579c37aeb8090ba62b46bd027
facebook-github-bot pushed a commit that referenced this pull request Jun 29, 2022
Summary: Pull Request resolved: #226

Reviewed By: Tianshu-Bao

Differential Revision: D34572815

fbshipit-source-id: 1f15b793535c5a2e6684186a03c9bc102b5d17a4
@wenleix wenleix deleted the export-D34572815 branch August 27, 2022 06:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants