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

Simplified-LTS : Convert synchronous ChunkStorage and ChunkMetadataStore API into async. #5067

Closed
sachin-j-joshi opened this issue Aug 13, 2020 · 0 comments · Fixed by #5200
Assignees
Labels
area/lts kind/enhancement Improvements that should be made version/0.9.0

Comments

@sachin-j-joshi
Copy link
Contributor

Problem description
To enable incremental approach and to avoid complexity a few APIs were implemented as synchronous APIs in initial implementation of SLTS.
This is a tracking issue for planned work to convert SLTS APIs to async version

Problem location

  • ChunkStorage interface and implementations
  • ChunkMetadataStore interface implementations

Suggestions for an improvement
Redefine the interface methods to return CompletableFuture<> . Refactor code as needed.

@sachin-j-joshi sachin-j-joshi self-assigned this Aug 13, 2020
@sachin-j-joshi sachin-j-joshi added area/lts kind/enhancement Improvements that should be made version/0.9.0 labels Aug 13, 2020
@sachin-j-joshi sachin-j-joshi changed the title Simplified-LTS : Convert synchronous ChunkStorage and ChunkMetadataStore methods into async. Simplified-LTS : Convert synchronous ChunkStorage and ChunkMetadataStore API into async. Aug 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/lts kind/enhancement Improvements that should be made version/0.9.0
Projects
None yet
1 participant