uima-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Adam Lally" <ala...@alum.rpi.edu>
Subject Re: [jira] Assigned: (UIMA-402) Adding Remote SOAP AE to Aggregate in CDE causes validation error
Date Thu, 14 Jun 2007 13:25:37 GMT
On 6/14/07, Marshall Schor <msa@schor.com> wrote:
> Adam Lally (JIRA) wrote:
> >      [ https://issues.apache.org/jira/browse/UIMA-402?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
> >
> > Adam Lally reassigned UIMA-402:
> > -------------------------------
> >
> >     Assignee: Marshall Schor  (was: Adam Lally)
> >
> > Marshall, I added the uima-core support for this.  Assigning to you to update the
CDE.
> >
> > I added a new method CasCreationUtils.mergeDelegateAnalysisEngineMetaData which
merges type systems, priorities, and indexes.  It also takes a parameter that will be populated
with information about
> > remote delegates that could not be contacted, rather than throwing an exception
in this case.
> >
> Adam, I see the CDE makes use of individual merge calls for the type
> system, the type priorities, and the index repositories.
>
> I examined the code to see if the CDE could switch to just one call that
> merged everything - it's fairly complex code, which I think would be
> risky to change.  So I'd like instead have versions of the individual
> part merge code.
>
> I can do the changes unless there's something un-obvious about this that
> I would miss?  Or, if you want to do this, let me know...
>
> -Marshall
>

I am trying to fight the explosion of the number of methods in
CasCreationUtils that all do similar things.  It's already a jungle in
there.  Also I'd rather encourage users to merge it all at once since
then you don't have to contact remotes more than once.

Maybe you can put the code that you need into the CDE rather than into
uimaj-core?

The easiest thing to do would be to just call
mergeDelegateAnalysisEngineMetaData and then extract the part you
need.

If you are concerned that this wouldn't perform well enough (say the
type system merge is complicated and you don't want to pay the price
each time you merge the fs indexes), then you could do a better
implementation where you copy the code from
CasCreationUtils.mergeDelegateAnalysisEngineMetaData and split it into
three different methods in the CDE.

-Adam

Mime
View raw message