Bug 496136 - [newindex] Implement a source (.java file) indexer for the new index
Summary: [newindex] Implement a source (.java file) indexer for the new index
Status: NEW
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 4.6   Edit
Hardware: PC Linux
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: JDT-Core-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 481796
Blocks: 149975
  Show dependency tree
 
Reported: 2016-06-14 15:23 EDT by Stefan Xenos CLA
Modified: 2018-05-10 13:22 EDT (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Stefan Xenos CLA 2016-06-14 15:23:40 EDT 
Currently the new index can only read .jar or .class files. We should also add support for indexing .java files. This will handle the case where the autobuilder is disabled.

In the case where the autobuilder is enabled, it probably makes sense to delay indexing (of .java files) until after the build completes and then index the derived .class files rather than the original .java files. This should speed up indexing considerably. In the case where the autobuilder is disabled, we should index the .java files directly.

In both cases, the NdResourceFile should point to the ".java" file as the source of truth for its fingerprint tests.
Comment 1 Stefan Xenos CLA 2016-06-20 16:03:14 EDT 
The implementation of the old indexer can be found in SourceElementParser and SourceIndexerRequestor.
Comment 2 Stefan Xenos CLA 2016-10-04 13:49:42 EDT 
Still an issue. Need to follow up.
Comment 3 Manoj N Palat CLA 2018-04-16 09:43:16 EDT 
bulk move out of 4.8