[YANGTOOLS-1116] Disconnect StmtNamespaceContext from StmtContext Created: 24/Jun/20 Updated: 28/Oct/20 Resolved: 01/Jul/20 |
|
| Status: | Resolved |
| Project: | yangtools |
| Component/s: | parser |
| Affects Version/s: | None |
| Fix Version/s: | 5.0.4, 4.0.11, 3.0.13 |
| Type: | Bug | Priority: | High |
| Reporter: | Robert Varga | Assignee: | Robert Varga |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
Screenshot_20200624_133713.png
|
||||||||||||
| Issue Links: |
|
||||||||||||
| Epic Link: | Parser Performance | ||||||||||||
| Description |
|
StmtNamespaceContext can be legally retained by AntlrYangXPathExpression, by which point it really should be a fixed prefix mapping for that instantiation. In particular we should not be retaining a SubstatementContext, as that ends up retaining large chunks of BuildGlobalContext. Examine lifecycle of affected codepaths and make sure we end up retaining a minimal context. |
| Comments |
| Comment by Robert Varga [ 29/Jun/20 ] |
|
Even with |