mbeckerle opened a new pull request #53: Change val and lazy val to def - 
multi-threading bug due to state?
URL: https://github.com/apache/incubator-daffodil/pull/53
   These two one-liner changes *may* have to do with the threading bug.
   It's hard to tell of course. The xml loader's xerces adapter is stateful and 
in theory cannot be shared across threads.  So while this may not be fixing 
"the" bug, it is fixing "some" bug. 
   Changing these two lines eliminates that incorrect sharing at the cost of 
just allocating objects. But this is per file loaded, so the overhead is not 
expected to be significant. 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:

With regards,
Apache Git Services

Reply via email to