jw3 edited a comment on pull request #520: URL: https://github.com/apache/daffodil/pull/520#issuecomment-821325697
Updated PR to push most of the implementation details down into the Schematron validator. Got rid of the raw stream concept, instead using the existing ability to pass in a config file to get the SVRL output path into the validator. Sending SVRL write errors back up as diagnostics. Much less surface area this way, but the config passing approach is not obvious. I think it will do for now, can be refined later by [refactoring the validator CLI arguments](https://github.com/apache/daffodil/pull/520#discussion_r608633044) to further hide some of these details, will add a ticket and follow up in another PR. Also the [refactoring of the java collections usage in the Validator API](https://github.com/apache/daffodil/pull/520#discussion_r609815124) will be followed up on. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
