[GitHub] [uima-uimaj] reckart opened a new pull request #155: [UIMA-6393]: Circular imports break resource manager cache

[GitBox <gi...@apache.org>]

reckart opened a new pull request #155:
URL: https://github.com/apache/uima-uimaj/pull/155


   - Added a minimal and a randomized test to reproduce the problem
   - Refactored import resolution algorithm. The types are no longer inlined in the cached type system descriptors. Instead, the import graph is walked on each call to `resolveImports()`. The imports are also no longer removed from the cached descriptors. This simplifies the algorithm and fixes the bug. Performance implications seem acceptable.
   - Type descriptors from imported type systems are defensively cloned into the resolved type system to avoid potential cache pollution
   - Added a several new tests
   - Import resolving mechanism was factored out into a separate `ImportResolver` class and is re-used for FSIndexCollections and TypePriorities
   - importUrlCache of the ResourceManager_impl is no longer used, has been deprecated and issues a warning when somebody tries to use it (was an internal detail so nobody should have been using it!)
   


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

To unsubscribe, e-mail: dev-unsubscribe@uima.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [uima-uimaj] reckart merged pull request #155: [UIMA-6393]: Circular imports break resource manager cache

[GitBox <gi...@apache.org>]

reckart merged pull request #155:
URL: https://github.com/apache/uima-uimaj/pull/155


   


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

To unsubscribe, e-mail: dev-unsubscribe@uima.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org