Matthieu,

Thanks for the explanation and my main concern would be,

I've seen a couple of usages that could be
changed to use the interface instead of the impl ..

Specifically, I think it would be helpful if we could change the following
impl references to the interface;

org.apache.ode.axis2.ODEServer ( line 82 )
org.apache.ode.axis2.ODEService ( line 55 )
org.apache.ode.jbi.OdeContext ( line 73 )

Thoughts?

My goal is to make sure we aren't missing anything in the interface ( and to
have a nice interface example ). Cory and I are beginning the work on a test
framework and I would like to get the public interfaces nailed down as much
as possible.

Also, I have note out to Maciej but if you have any input on the following
methods added to the BpelServer interface ...

 void registerBpelEventListener(BpelEventListener bpelEventListener);
 void registerMessageExchangeInterceptor(MessageExchangeInterceptor
interceptor);

Thanks,

Lance

On 8/11/06, Matthieu Riou < [EMAIL PROTECTED]> wrote:

Hi Lance,

The IL is instantiating the BpelServer (in the IL server classes) so it
needs to import the BpelServerImpl to at least instantiate it and set a
couple of needed attributes. I've seen a couple of usages that could be
changed to use the interface instead of the impl, so to clean up a bit,
but
that still won't remove the dependency. The IL is the one starting the
BPEL
server.

Cheers,

Matthieu

On 8/10/06, Lance Waterman < [EMAIL PROTECTED]> wrote:
>
> Maciej, Matthieu;
>
> I see that the reference implementations of the IL are using
> BpelServerImpl
> rather than the BpelServer interface and am curious as to why?
>
>
> Thanks,
>
> Lance
>
>


Reply via email to