You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@daffodil.apache.org by GitBox <gi...@apache.org> on 2018/03/06 21:25:06 UTC

[GitHub] mbeckerle opened a new pull request #53: Change val and lazy val to def - multi-threading bug due to state?

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. 
   
   DAFFODIL-1908

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services