Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 329976 - [index] Think about renaming some concepts
Summary: [index] Think about renaming some concepts
Status: CLOSED WONTFIX
Alias: None
Product: TMF
Classification: Modeling
Component: Xtext (show other bugs)
Version: 2.0.0   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-11-11 04:16 EST by Jan Koehnlein CLA
Modified: 2017-09-19 18:03 EDT (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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