[MDSAL-611] Cache NodeIdentifier lookup in QueryBuilderState.bindMethod() Created: 26/Oct/20 Updated: 26/Oct/20 Resolved: 26/Oct/20 |
|
| Status: | Resolved |
| Project: | mdsal |
| Component/s: | Binding runtime |
| Affects Version/s: | None |
| Fix Version/s: | 7.0.2 |
| Type: | Improvement | Priority: | Medium |
| Reporter: | Robert Varga | Assignee: | Robert Varga |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Epic Link: | Searchable MD-SAL |
| Description |
|
When we are building a query, we end up creating a NodeIdentifier for the found leaf. Finding the leaf involves quite a few string operations, hence we want to side-step it. Furthermore, once we have located the node we follow up with a caching NodeIdentifier.create(), which could also be cached. Add a cache to DefaultQueryBuilder, so that all queries with particular code end up reusing a common cache of these. |