click-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Md. Jahid Shohel" <development....@gmail.com>
Subject Re: [jira] Updated: (CLK-661) Add Java based configuration
Date Thu, 17 Jun 2010 19:04:28 GMT
Finn,

It looks great, actually that's how it should be.

But it will create problem for different situations. If I want to integrate
my annotation extension with this, then it will still force me to extend
XmlConfigService. Because there is a sequence issue. Let me explain how-

Say when we are booting up the application we have to go through "load
mode", "load common headers", "load pages"steps. And lets consider that
there are two services XmlConfigService and AnnotationConfigService.

And now about the sequence issue, when XmlConfigService is loading pages, it
will call page interceptors. Now before this page interceptors are called,
all the pages which are using annotation, they need to be processed.

Second example, before we process pages, we need to load autobinding,
automapping configurations. Now if someone did not configure his autobinding
and automapping on xml, but configured on annotation, then loading pages
will fail.

There are other situations ( i exactly can't remember all of them right now)
where this situation will occur.

So, once your patch is committed, I want to work on this issue, so that this
sequence problem is solved. I have the clear picture about the problem,
because I had to went through each of them when I was creating annotation
based configuration.

If you, or anyone else do not agree with me, or I have a misunderstanding
please let me know.

Regards,

Jahid

On Thu, Jun 17, 2010 at 4:49 PM, Finn Bock <bckfnn@gmail.com> wrote:

> It is attached to CLK-661 as a patch, you can also take a look at the
> DefaultConfigService class here:
>
> http://people.apache.org/~bckfnn/DefaultConfigService.java<http://people.apache.org/%7Ebckfnn/DefaultConfigService.java>
>
> regards,
> Finn
>

Mime
View raw message