Compiler space/speed issue is *solved* to my satisfaction. All tests pass. I also tested all DFDLSchemas from github, and all our FOUO schemas and they all work also.
The PR is ready for "real" review now. I removed the "WIP" status on it. I have not yet updated the design notes to match the implementation. The pending fixes to isHidden, and to namespace binding minimization - basically those fixes are needed, but we have no tests nor schemas that run into the problems. E.g, for isHidden: we have no tests nor schemas where a single group is both hidden and non-hidden. Similarly we have no tests nor schemas where namespace binding minimization must be different on two different inclusions of a common element. We could *not fix* those issues and create JIRA tickets for lifting them. But we need tests to at least exercise the problems, so we can characterize what happens if we do these unimplemented things. ________________________________ From: Beckerle, Mike Sent: Thursday, February 13, 2020 10:29 PM To: [email protected] <[email protected]> Subject: progress on schema compiler space/speed issue See https://github.com/apache/incubator-daffodil/pull/324 I still have debugging to do, and I'm not happy with the non-type-safety, and maintainability headaches this creates, but I was able to get rid of the combinatorial explosion with a modest number of lines of code changed. But most schemas work now, and schema compilation is faster, and is seemingly linear in schema size with these changes.
