google-code-export / uimafit

Automatically exported from code.google.com/p/uimafit
2 stars 1 forks source link

SimplePipeline should use AAEs to run multi-step pipelines #115

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
SimplePipeline currently uses for-loops to process multi-step pipelines. 
Instead, it should warp all AEs from a multi-step pipeline into a AAE and just 
call process() on that. This should resolve problems with CAS multipliers and 
mergers as well as with shared resources being instantiated multiple times when 
used be different AEs.

This change should be compatible as no API change should be required.

Original issue reported on code.google.com by richard.eckart on 19 Mar 2012 at 9:16

GoogleCodeExporter commented 9 years ago
This will only work for those methods that accept AE descriptors. It's not 
possible to wrap already instantiated AEs into a AAE.

Original comment by richard.eckart on 19 Mar 2012 at 9:17

GoogleCodeExporter commented 9 years ago
I changed SimplePipeline to use an internal AAE where possible and added a test 
case for properly shared external resources to ExternalResourceFactoryTest.
---
Committed revision 803.

Original comment by richard.eckart on 19 Mar 2012 at 9:53

GoogleCodeExporter commented 9 years ago

Original comment by richard.eckart on 24 Mar 2012 at 5:10