[YANGTOOLS-715] Parser: do not ignore SourceExceptions Created: 12/Nov/16 Updated: 10/Apr/22 Resolved: 18/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: | 7161 | ||||||||
| Description |
|
YangStatementParserListenerImpl currently catches SourceExceptions and LOG.warn()s about them, but does not propagate them. This means that source-level problems may get ignored with nothing but a warning, potentially leading to incorrect models. Fix this by not catching SourceException and letting the reactor deal with them. |
| Comments |
| Comment by Robert Varga [ 12/Nov/16 ] |
| Comment by Robert Varga [ 12/Nov/16 ] |
|
This needs a weather item and a test-drive autorelease. |
| Comment by Robert Varga [ 14/Nov/16 ] |
|
Autorelease -Pq does not show a regression. netconf and mdsal continue to build successfully. Weather item has been posted: https://lists.opendaylight.org/pipermail/release/2016-November/008755.html. |