[jira] Commented: (AXIS2-938) Plugpoint to allow for migration between context and TLS

2006-08-01 Thread Davanum Srinivas (JIRA)
[ http://issues.apache.org/jira/browse/AXIS2-938?page=comments#action_12424963 ] Davanum Srinivas commented on AXIS2-938: Looks good Bill. Thanks. -- dims Plugpoint to allow for migration between context and TLS

[jira] Commented: (AXIS2-938) Plugpoint to allow for migration between context and TLS

2006-07-31 Thread Bill Nagy (JIRA)
[ http://issues.apache.org/jira/browse/AXIS2-938?page=comments#action_12424587 ] Bill Nagy commented on AXIS2-938: - (Sorry that I'm a little late to the discussion; I just switched machines and apparently my mail filtering is screwed up.) The

Re: [jira] Commented: (AXIS2-938) Plugpoint to allow for migration between context and TLS

2006-07-31 Thread Sanjiva Weerawarana
On Mon, 2006-07-31 at 10:41 -0700, Bill Nagy (JIRA) wrote: (Sorry that I'm a little late to the discussion; I just switched machines and apparently my mail filtering is screwed up.) The plugpoint was indeed meant to be PM-agnostic, and I know for a fact that when Tuscany integrates with

Re: [jira] Commented: (AXIS2-938) Plugpoint to allow for migration between context and TLS

2006-07-31 Thread Davanum Srinivas
Bill, If we decide to mode the whole thing into modules/core, then i will have no problem keeping the methods for migrators and make it similar to modules, transports, phases etc. Here's my problem, the Patch does not reflect the intention. If you check it in as-is right now. Anyone using the

Re: [jira] Commented: (AXIS2-938) Plugpoint to allow for migration between context and TLS

2006-07-31 Thread Bill Nagy
Hi dims, If all you are asking me for is to add calls into the other MessageReceiver's and the ServiceClient, I'd be happy to do it; I just didn't have a use case that required that, so didn't want to clutter the code. Alternatively I can make it more explicit in the comments that it is up to

Re: [jira] Commented: (AXIS2-938) Plugpoint to allow for migration between context and TLS

2006-07-31 Thread Bill Nagy
Hi Sanjiva, Unless I'm not explaining it well, the only piece that is different from what I had described earlier (http://marc.theaimsgroup.com/?l=axis-devm=115270290506219w=2) is the interface; there are now two extra methods to make it more clear to someone who is writing a migrator (or reading

Re: [jira] Commented: (AXIS2-938) Plugpoint to allow for migration between context and TLS

2006-07-31 Thread Davanum Srinivas
I am officially confused...But let's start with: - explicit comments as mentioned in your email. - Please move the methods in AxisConfiguration to ConfigurationContext as AxisConfiguration is only till deployment. - Ok to leave the explicit methods, would prefer get/set property till we get this

[jira] Commented: (AXIS2-938) Plugpoint to allow for migration between context and TLS

2006-07-29 Thread Rich Scheuerle (JIRA)
[ http://issues.apache.org/jira/browse/AXIS2-938?page=comments#action_12424290 ] Rich Scheuerle commented on AXIS2-938: -- I read the code and Dim's comments. My main concern is that the plugpoint is useful for any programming model that

[jira] Commented: (AXIS2-938) Plugpoint to allow for migration between context and TLS

2006-07-29 Thread Nick Gallardo (JIRA)
[ http://issues.apache.org/jira/browse/AXIS2-938?page=comments#action_12424293 ] Nick Gallardo commented on AXIS2-938: - Rich, Is your concern mostly just in which module the code lives? Or is it more where the code is called from?

[jira] Commented: (AXIS2-938) Plugpoint to allow for migration between context and TLS

2006-07-29 Thread Davanum Srinivas (JIRA)
[ http://issues.apache.org/jira/browse/AXIS2-938?page=comments#action_12424299 ] Davanum Srinivas commented on AXIS2-938: Rich, It would be useful if the hooks were called from core module, right now it isn't. All the calls to the

[jira] Commented: (AXIS2-938) Plugpoint to allow for migration between context and TLS

2006-07-27 Thread Davanum Srinivas (JIRA)
[ http://issues.apache.org/jira/browse/AXIS2-938?page=comments#action_12423995 ] Davanum Srinivas commented on AXIS2-938: Here's my train of thought: - 3 changes to classes under jaxws look ok. - 3 new classes (ThreadContext*.java) look