Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 329976

Summary: [index] Think about renaming some concepts
Product: [Modeling] TMF Reporter: Jan Koehnlein <jan>
Component: XtextAssignee: Project Inbox <tmf.xtext-inbox>
Status: CLOSED WONTFIX QA Contact:
Severity: enhancement    
Priority: P3 CC: chanskw
Version: 2.0.0   
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:

Description Jan Koehnlein CLA 2010-11-11 04:16:40 EST
I'd like to open a discussion on the naming of some classes in the index API.
I know that this is a central API and renaming could break lot of things, but working with the index data model sometimes becomes a bit tedious due to the naming.

1) IResourceDescription, IReferenceDescripton, and IResourceDescriptions look almost the same (did you notice the difference while reading this?) and it is likely they are all used within the same context. This becomes especially annoying when using code assist.

2) The term 'index' does not appear in any name of the framework classes nor packages even though we use it a lot when talking about that API.

3) While its purpose is to store the configuration of all IContainers, the most prominent purpose of the IAllContainersState for our clients is to manage container (project) dependencies. Given that, this interface is pretty hard to find.

4) IResourceDescription.Manager is a factory and a dependency manager. From my experience, clients mostly touch it as the entry hook to configure what is indexed. Coming from that side I wouldn't find it. Maybe we should extract the factory part to a IResourceDescription.Factory.

Suggestions
- IResourceDescription stay the same
- IReferenceDescription -> ICrossReferenceDescription 
- IResourceDescription -> IIndex(State)
- IAllContainersState -> IContainerIndex

What do you think?
Comment 1 Jan Koehnlein CLA 2011-11-11 09:36:14 EST
We don't plan any major release in the near future. In the meantime, I got auite used to the names. Closing as wont fix.
Comment 2 Karsten Thoms CLA 2017-09-19 17:52:47 EDT
Closing all bugs that were set to RESOLVED before Neon.0
Comment 3 Karsten Thoms CLA 2017-09-19 18:03:30 EDT
Closing all bugs that were set to RESOLVED before Neon.0