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

Deprecate s3 file system #47046

Merged
merged 1 commit into from
Feb 12, 2021
Merged

Conversation

yongtang
Copy link
Member

@yongtang yongtang commented Feb 9, 2021

This PR is part of the effort to switch to modular file system support
by deprecates s3 file system and direct user to use modular file system
from tensorflow-io instead.

A TF_ENABLE_LEGACY_FILESYSTEM=1 will allow the usage of legacy hdfs file
system the same way as before (a warning will be displayed).

/cc @mihaimaruseac @vnvo2409 @tensorflow/sig-io-maintainers @burgerkingeater

Signed-off-by: Yong Tang [email protected]

This PR is part of the effort to switch to modular file system support
by deprecates s3 file system and direct user to use modular file system
from tensorflow-io instead.

A `TF_ENABLE_LEGACY_FILESYSTEM=1` will allow the usage of legacy hdfs file
system the same way as before (a warning will be displayed).

Signed-off-by: Yong Tang <[email protected]>
@google-ml-butler google-ml-butler bot added the size:XS CL Change Size: Extra Small label Feb 9, 2021
@google-cla google-cla bot added the cla: yes label Feb 9, 2021
@google-ml-butler google-ml-butler bot added kokoro:force-run Tests on submitted change ready to pull PR ready for merge process labels Feb 9, 2021
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Feb 9, 2021
@yongtang
Copy link
Member Author

yongtang commented Feb 9, 2021

Also cc @mihaimaruseac @vnvo2409

@google-ml-butler google-ml-butler bot added the kokoro:force-run Tests on submitted change label Feb 9, 2021
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Feb 9, 2021
@rthadur rthadur removed the ready to pull PR ready for merge process label Feb 9, 2021
@gbaned gbaned self-assigned this Feb 10, 2021
@gbaned gbaned added the comp:core issues related to core part of tensorflow label Feb 10, 2021
@google-ml-butler google-ml-butler bot added kokoro:force-run Tests on submitted change ready to pull PR ready for merge process labels Feb 12, 2021
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Feb 12, 2021
@copybara-service copybara-service bot merged commit 0acbcc5 into tensorflow:master Feb 12, 2021
@yongtang yongtang deleted the deprecate-aws branch February 12, 2021 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes comp:core issues related to core part of tensorflow ready to pull PR ready for merge process size:XS CL Change Size: Extra Small
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants