[YANGTOOLS-707] Reimplement YinStatementSourceImpl in terms of DOM Created: 27/Oct/16 Updated: 10/Apr/22 Resolved: 05/Nov/16 |
|
| Status: | Resolved |
| Project: | yangtools |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | ||
| Reporter: | Robert Varga | Assignee: | Robert Varga |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: All |
||
| Issue Links: |
|
||||||||||||
| External issue ID: | 7054 | ||||||||||||
| Description |
|
Current implementation of YIN parsing is based on StAX, which means that each processing phase ends up re-parsing the XML from scratch. It also means that YinStatementParserImpl is very complicated. Using a W3C DOM Document is much more natural, as that will end up parsing the input XML only once and provides convenient iteration, leading to a much simpler and performant implementation at the cost of some memory overhead. |
| Comments |
| Comment by Robert Varga [ 27/Oct/16 ] |
|
Refactoring current implementation to make it call startElement with its argument ready would be a pain, but will become trivial with this piece implemented. |
| Comment by Robert Varga [ 30/Oct/16 ] |
|
master: |