eclipse-xtext / eclipse-xtext/xtext
Pass on load options to ResourceStorage* classes
Nobody has claimed this yet.
- Dominant language
- Java
- Stars
- 831
- Forks
- 330
- Avg merge
- 3d 7h
- Merged PRs (30d)
- 12
Description
I think that StorageAwareResource#load(Map<?, ?>) should pass on the load options Map to the ResourceStorageFacade and ResourceStorageLoadable. This for example allows implementations to implement conditional loading.
My use case is to implement on-demand loading of the node model and also completely suppress loading of the node model. The reason being that the node model typically requires a lot of memory and is in many cases not required.
For symmetry it would probably also make sense to pass an options Map to ResourceStorageWritable.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with StorageAwareResource#load(Map), then inspect the ResourceStorageFacade, ResourceStorageLoadable, and ResourceStorageWritable entry points named in the issue. Determine how load options should flow through these APIs, including the writable symmetry, and consider how conditional or suppressed loading should be represented. Done means the relevant storage implementations can receive the options map.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- tooling
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100