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

Drop the heavy use of Properties all over the place and replace it with the S3FileSystemConfiguration #196

Open
4 tasks
carlspring opened this issue Feb 1, 2021 · 0 comments

Comments

@carlspring
Copy link
Owner

carlspring commented Feb 1, 2021

Task Description

Once #195 has been implemented, we need to refactor the code for the S3FileSystemProvider and S3FileSystem as necessary so that we can drop the heavy use of properties.

Tasks

The following tasks will need to be carried out:

  • Assess what will need to be done and update this task, if and as necessary.
  • Re-work the S3FileSystem class so that it:
    • Contains a registry (map) of all the file system instances per bucket.
    • Has a constructor that only works with S3FileSystemConfiguration.
  • Carry out the refactoring.
  • Fix any broken tests.

Task Relationships

This task:

Help

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant