[OAK-8141] Replace String path with custom data type

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.14.0
• Component/s: documentmk
• Assignee: Marcel Reutegger
• Reporter: Marcel Reutegger
• Created: 19/Mar/19 13:41
• Updated: 12/Nov/19 12:11
• Resolved: 10/Apr/19 11:14

Description

DocumentNodeState objects and other related data is currently identified with a String path in the various caches in the DocumentNodeStore. Read benchmarks like GetDeepNodeTest that only read data from memory are significantly slower on a DocumentNodeStore compared to a SegmentNodeStore. In these kind of tests the DocumentNodeStore is usually busy with String operations like constructing the path for a child node or calculating the hash code for a lookup in a cache.

This issue is about a potential improvement that replaces the use of a plain String for the path of a DocumentNodeState and use a custom Path type instead.

Issue Links

• causes: OAK-8758 tests make "equals" comparison of Path and String
• is related to: OAK-8457 Constructor allows creating invalid path