Skip to content

[DataPipe] Use alternate reading mode for streams in TarArchiveLoader#653

Closed
NivekT wants to merge 1 commit intogh/NivekT/88/basefrom
gh/NivekT/88/head
Closed

[DataPipe] Use alternate reading mode for streams in TarArchiveLoader#653
NivekT wants to merge 1 commit intogh/NivekT/88/basefrom
gh/NivekT/88/head

Conversation

@NivekT
Copy link
Contributor

@NivekT NivekT commented Jul 13, 2022

NivekT added a commit that referenced this pull request Jul 13, 2022
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jul 13, 2022
@NivekT NivekT changed the title [DataPipe] use alternate reading mode for streams in TarArchiveLoader [DataPipe] Use alternate reading mode for streams in TarArchiveLoader Jul 13, 2022
@NivekT NivekT requested review from VitalyFedyunin and ejguan July 13, 2022 22:04
Copy link
Contributor

@ejguan ejguan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, LGTM

@NivekT
Copy link
Contributor Author

NivekT commented Jul 14, 2022

@NivekT has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot facebook-github-bot deleted the gh/NivekT/88/head branch July 18, 2022 14: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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants