[jira] Commented: (GERONIMO-3124) Reminder: Switch Axis2 stack back to axis2-saaj from Sun SAAJ Impl

2007-05-21 Thread Ted Kirby (JIRA)

[ 
https://issues.apache.org/jira/browse/GERONIMO-3124?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12497693
 ] 

Ted Kirby commented on GERONIMO-3124:
-

These configs have 
org.apache.geronimo.configs/webservices-common/2.0-SNAPSHOT/car as a parent:
org.apache.geronimo.configs/openejb/2.0-SNAPSHOT/car
org.apache.geronimo.configs/axis/2.0-SNAPSHOT/car
org.apache.geronimo.configs/tomcat6/2.0-SNAPSHOT/car
org.apache.geronimo.configs/jaxws-deployer/2.0-SNAPSHOT/car
org.apache.geronimo.configs/axis2/2.0-SNAPSHOT/car

These configs have axis2-saaj in their classpath:
org.apache.geronimo.configs/jee-specs/2.0-SNAPSHOT/car
org.apache.geronimo.configs/axis2/2.0-SNAPSHOT/car

Thus, axis2 is only used for jee-specs and axis2, whereas saaj-impl will be 
used for openejb, axis, tomcat6 and jaxws-deployer.  saaj-impl refers to 
internal sun classes only defined with a Sun JDK, thus cannot be used with any 
other JDK, and therefore is not a good candidate for use as a default in 
webservices-common.

> Reminder:  Switch Axis2 stack back to axis2-saaj from Sun SAAJ Impl
> ---
>
> Key: GERONIMO-3124
> URL: https://issues.apache.org/jira/browse/GERONIMO-3124
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>  Components: webservices
>Affects Versions: 2.0-M6
>Reporter: Donald Woods
> Assigned To: Lin Sun
>Priority: Critical
> Fix For: 2.0-M6
>
> Attachments: G3124.patch
>
>
> This is a placeholder, so we don't forget to rollback the temporary changes 
> made to use the Sun SAAJ Impl instead of the axis2-saaj jar.
> Most of the following changes will have to be rolled back - Revisions 530234, 
> 531260
> Author: dims
> Date: Wed Apr 18 19:18:50 2007
> New Revision: 530234
> URL: http://svn.apache.org/viewvc?view=rev&rev=530234
> Log:
> Switch Axis2 over to sun's saaj jars as well. Once we stabilize, we can 
> switch back to see what breaks with Axis2's saaj impl jars
> Modified:
> geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml
> geronimo/server/trunk/configs/axis2/pom.xml
> 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
> Modified: geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml
> URL: 
> http://svn.apache.org/viewvc/geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml?view=diff&rev=530234&r1=530233&r2=530234
> ==
> --- geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml (original)
> +++ geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml Wed Apr 18 
> 19:18:50 2007
> @@ -83,19 +83,11 @@
>  
>  
>  org.apache.axis2
> -axis2-saaj
> Modified: geronimo/server/trunk/configs/axis2/pom.xml
> URL: 
> http://svn.apache.org/viewvc/geronimo/server/trunk/configs/axis2/pom.xml?view=diff&rev=530234&r1=530233&r2=530234
> ==
> --- geronimo/server/trunk/configs/axis2/pom.xml (original)
> +++ geronimo/server/trunk/configs/axis2/pom.xml Wed Apr 18 19:18:50 2007
> @@ -110,19 +110,11 @@
>  
>  
>  org.apache.axis2
> -axis2-saaj
> -
> Modified: 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
> URL: 
> http://svn.apache.org/viewvc/geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java?view=diff&rev=530234&r1=530233&r2=530234
> ==
> --- 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
>  (original)
> +++ 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
>  Wed Apr 18 19:18:50 2007
> @@ -32,12 +32,12 @@
>
> "org.apache.axis.soap.SOAPFactoryImpl",
>
> "org.apache.axis.soap.SOAPConnectionFactoryImpl",
>
> "org.apache.axis.soap.SAAJMetaFactoryImpl"));
> -SAAJ_FACTORIES.put(SAAJUniverse.Type.AXIS2.toString(), 
> -   
> createSAAJInfo("org.apache.axis2.saaj.MessageFactoryImpl", 
> -  
> "org.apache.axis2.saaj.SOAPFactoryImpl", 
> -  
> "org.apache.axis2.saaj.SOAPConnectionFactoryImpl", 
> -   

[jira] Commented: (GERONIMO-3124) Reminder: Switch Axis2 stack back to axis2-saaj from Sun SAAJ Impl

2007-05-21 Thread Lin Sun (JIRA)

[ 
https://issues.apache.org/jira/browse/GERONIMO-3124?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12497691
 ] 

Lin Sun commented on GERONIMO-3124:
---

Jarek, I think this can be an issue for certain jaxr scenarios (if saaj-impl is 
needed) with axis2 as the web service provider, before the system performs some 
axis2 deployment to start the axis2 config module.  What do you think?  Lin

> Reminder:  Switch Axis2 stack back to axis2-saaj from Sun SAAJ Impl
> ---
>
> Key: GERONIMO-3124
> URL: https://issues.apache.org/jira/browse/GERONIMO-3124
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>  Components: webservices
>Affects Versions: 2.0-M6
>Reporter: Donald Woods
> Assigned To: Lin Sun
>Priority: Critical
> Fix For: 2.0-M6
>
> Attachments: G3124.patch
>
>
> This is a placeholder, so we don't forget to rollback the temporary changes 
> made to use the Sun SAAJ Impl instead of the axis2-saaj jar.
> Most of the following changes will have to be rolled back - Revisions 530234, 
> 531260
> Author: dims
> Date: Wed Apr 18 19:18:50 2007
> New Revision: 530234
> URL: http://svn.apache.org/viewvc?view=rev&rev=530234
> Log:
> Switch Axis2 over to sun's saaj jars as well. Once we stabilize, we can 
> switch back to see what breaks with Axis2's saaj impl jars
> Modified:
> geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml
> geronimo/server/trunk/configs/axis2/pom.xml
> 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
> Modified: geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml
> URL: 
> http://svn.apache.org/viewvc/geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml?view=diff&rev=530234&r1=530233&r2=530234
> ==
> --- geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml (original)
> +++ geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml Wed Apr 18 
> 19:18:50 2007
> @@ -83,19 +83,11 @@
>  
>  
>  org.apache.axis2
> -axis2-saaj
> Modified: geronimo/server/trunk/configs/axis2/pom.xml
> URL: 
> http://svn.apache.org/viewvc/geronimo/server/trunk/configs/axis2/pom.xml?view=diff&rev=530234&r1=530233&r2=530234
> ==
> --- geronimo/server/trunk/configs/axis2/pom.xml (original)
> +++ geronimo/server/trunk/configs/axis2/pom.xml Wed Apr 18 19:18:50 2007
> @@ -110,19 +110,11 @@
>  
>  
>  org.apache.axis2
> -axis2-saaj
> -
> Modified: 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
> URL: 
> http://svn.apache.org/viewvc/geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java?view=diff&rev=530234&r1=530233&r2=530234
> ==
> --- 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
>  (original)
> +++ 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
>  Wed Apr 18 19:18:50 2007
> @@ -32,12 +32,12 @@
>
> "org.apache.axis.soap.SOAPFactoryImpl",
>
> "org.apache.axis.soap.SOAPConnectionFactoryImpl",
>
> "org.apache.axis.soap.SAAJMetaFactoryImpl"));
> -SAAJ_FACTORIES.put(SAAJUniverse.Type.AXIS2.toString(), 
> -   
> createSAAJInfo("org.apache.axis2.saaj.MessageFactoryImpl", 
> -  
> "org.apache.axis2.saaj.SOAPFactoryImpl", 
> -  
> "org.apache.axis2.saaj.SOAPConnectionFactoryImpl", 
> -  
> "org.apache.axis2.saaj.SAAJMetaFactoryImpl"));
> -SAAJ_FACTORIES.put(SAAJUniverse.Type.SUN.toString(), 
> +SAAJ_FACTORIES.put(SAAJUniverse.Type.AXIS2.toString(),
> +   
> createSAAJInfo("com.sun.xml.messaging.saaj.soap.ver1_1.SOAPMessageFactory1_1Impl",
> +  
> "com.sun.xml.messaging.saaj.soap.ver1_1.SOAPFactory1_1Impl",
> +  
> "com.sun.xml.messaging.saaj.client.p2p.HttpSOAPConnectionFactory",
> +  
> "com.sun.xml.messaging.saaj.s

[jira] Commented: (GERONIMO-3124) Reminder: Switch Axis2 stack back to axis2-saaj from Sun SAAJ Impl

2007-05-21 Thread Lin Sun (JIRA)

[ 
https://issues.apache.org/jira/browse/GERONIMO-3124?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12497679
 ] 

Lin Sun commented on GERONIMO-3124:
---

If tomcat + axis2 assembly is used, then axis2 deployer config 
(org.apache.geronimo.configs/axis2/2.0-SNAPSHOT/car) should be started.
Axis2's saaj impl is specified as a depedency in axis2 config's pom.xml so it 
should be loaded wehenever axis2 config is started.  Axis2 config is started 
whenever an axis2 webservice deployment is performed (Axis2 deployer config 
depends on Axis2 config).   Does your isAxis2InClassLoader() checking happen 
before any deployment?   


> Reminder:  Switch Axis2 stack back to axis2-saaj from Sun SAAJ Impl
> ---
>
> Key: GERONIMO-3124
> URL: https://issues.apache.org/jira/browse/GERONIMO-3124
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>  Components: webservices
>Affects Versions: 2.0-M6
>Reporter: Donald Woods
> Assigned To: Lin Sun
>Priority: Critical
> Fix For: 2.0-M6
>
> Attachments: G3124.patch
>
>
> This is a placeholder, so we don't forget to rollback the temporary changes 
> made to use the Sun SAAJ Impl instead of the axis2-saaj jar.
> Most of the following changes will have to be rolled back - Revisions 530234, 
> 531260
> Author: dims
> Date: Wed Apr 18 19:18:50 2007
> New Revision: 530234
> URL: http://svn.apache.org/viewvc?view=rev&rev=530234
> Log:
> Switch Axis2 over to sun's saaj jars as well. Once we stabilize, we can 
> switch back to see what breaks with Axis2's saaj impl jars
> Modified:
> geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml
> geronimo/server/trunk/configs/axis2/pom.xml
> 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
> Modified: geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml
> URL: 
> http://svn.apache.org/viewvc/geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml?view=diff&rev=530234&r1=530233&r2=530234
> ==
> --- geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml (original)
> +++ geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml Wed Apr 18 
> 19:18:50 2007
> @@ -83,19 +83,11 @@
>  
>  
>  org.apache.axis2
> -axis2-saaj
> Modified: geronimo/server/trunk/configs/axis2/pom.xml
> URL: 
> http://svn.apache.org/viewvc/geronimo/server/trunk/configs/axis2/pom.xml?view=diff&rev=530234&r1=530233&r2=530234
> ==
> --- geronimo/server/trunk/configs/axis2/pom.xml (original)
> +++ geronimo/server/trunk/configs/axis2/pom.xml Wed Apr 18 19:18:50 2007
> @@ -110,19 +110,11 @@
>  
>  
>  org.apache.axis2
> -axis2-saaj
> -
> Modified: 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
> URL: 
> http://svn.apache.org/viewvc/geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java?view=diff&rev=530234&r1=530233&r2=530234
> ==
> --- 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
>  (original)
> +++ 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
>  Wed Apr 18 19:18:50 2007
> @@ -32,12 +32,12 @@
>
> "org.apache.axis.soap.SOAPFactoryImpl",
>
> "org.apache.axis.soap.SOAPConnectionFactoryImpl",
>
> "org.apache.axis.soap.SAAJMetaFactoryImpl"));
> -SAAJ_FACTORIES.put(SAAJUniverse.Type.AXIS2.toString(), 
> -   
> createSAAJInfo("org.apache.axis2.saaj.MessageFactoryImpl", 
> -  
> "org.apache.axis2.saaj.SOAPFactoryImpl", 
> -  
> "org.apache.axis2.saaj.SOAPConnectionFactoryImpl", 
> -  
> "org.apache.axis2.saaj.SAAJMetaFactoryImpl"));
> -SAAJ_FACTORIES.put(SAAJUniverse.Type.SUN.toString(), 
> +SAAJ_FACTORIES.put(SAAJUniverse.Type.AXIS2.toString(),
> +   
> createSAAJInfo("com.sun.xml.messaging.saaj.soap.ver1_1.SOAPMessageFactory1_1Impl",
> +  
> "com.sun.xml

[jira] Commented: (GERONIMO-3124) Reminder: Switch Axis2 stack back to axis2-saaj from Sun SAAJ Impl

2007-05-21 Thread Donald Woods (JIRA)

[ 
https://issues.apache.org/jira/browse/GERONIMO-3124?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12497671
 ] 

Donald Woods commented on GERONIMO-3124:


Well, we need to seriously consider switching over to the Axis2 SAAJ, given the 
Sun RI version has hard coded depends to classes that are only provided in the 
Sun JVM.  Otherwise, webservices in Geronimo 2.0 will never work on other JVMs, 
like the ones from IBM, BEA, Apache Harmony, 

Also, I don't see how the Axis SAAJ jar would ever get loaded on the classpath, 
given the webservices-common code (common to both CXF and Axis2) loads the Sun 
RI jar

Ted, can you post the classpath you are seeing, when trying to run the 
Tomcat+Axis2 assembly?


> Reminder:  Switch Axis2 stack back to axis2-saaj from Sun SAAJ Impl
> ---
>
> Key: GERONIMO-3124
> URL: https://issues.apache.org/jira/browse/GERONIMO-3124
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>  Components: webservices
>Affects Versions: 2.0-M6
>Reporter: Donald Woods
> Assigned To: Lin Sun
>Priority: Critical
> Fix For: 2.0-M6
>
> Attachments: G3124.patch
>
>
> This is a placeholder, so we don't forget to rollback the temporary changes 
> made to use the Sun SAAJ Impl instead of the axis2-saaj jar.
> Most of the following changes will have to be rolled back - Revisions 530234, 
> 531260
> Author: dims
> Date: Wed Apr 18 19:18:50 2007
> New Revision: 530234
> URL: http://svn.apache.org/viewvc?view=rev&rev=530234
> Log:
> Switch Axis2 over to sun's saaj jars as well. Once we stabilize, we can 
> switch back to see what breaks with Axis2's saaj impl jars
> Modified:
> geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml
> geronimo/server/trunk/configs/axis2/pom.xml
> 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
> Modified: geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml
> URL: 
> http://svn.apache.org/viewvc/geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml?view=diff&rev=530234&r1=530233&r2=530234
> ==
> --- geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml (original)
> +++ geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml Wed Apr 18 
> 19:18:50 2007
> @@ -83,19 +83,11 @@
>  
>  
>  org.apache.axis2
> -axis2-saaj
> Modified: geronimo/server/trunk/configs/axis2/pom.xml
> URL: 
> http://svn.apache.org/viewvc/geronimo/server/trunk/configs/axis2/pom.xml?view=diff&rev=530234&r1=530233&r2=530234
> ==
> --- geronimo/server/trunk/configs/axis2/pom.xml (original)
> +++ geronimo/server/trunk/configs/axis2/pom.xml Wed Apr 18 19:18:50 2007
> @@ -110,19 +110,11 @@
>  
>  
>  org.apache.axis2
> -axis2-saaj
> -
> Modified: 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
> URL: 
> http://svn.apache.org/viewvc/geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java?view=diff&rev=530234&r1=530233&r2=530234
> ==
> --- 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
>  (original)
> +++ 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
>  Wed Apr 18 19:18:50 2007
> @@ -32,12 +32,12 @@
>
> "org.apache.axis.soap.SOAPFactoryImpl",
>
> "org.apache.axis.soap.SOAPConnectionFactoryImpl",
>
> "org.apache.axis.soap.SAAJMetaFactoryImpl"));
> -SAAJ_FACTORIES.put(SAAJUniverse.Type.AXIS2.toString(), 
> -   
> createSAAJInfo("org.apache.axis2.saaj.MessageFactoryImpl", 
> -  
> "org.apache.axis2.saaj.SOAPFactoryImpl", 
> -  
> "org.apache.axis2.saaj.SOAPConnectionFactoryImpl", 
> -  
> "org.apache.axis2.saaj.SAAJMetaFactoryImpl"));
> -SAAJ_FACTORIES.put(SAAJUniverse.Type.SUN.toString(), 
> +SAAJ_FACTORIES.put(SAAJUniverse.Type.AXIS2.toString(),
> +   
> createSAAJInfo("com.sun.xml.messaging.saaj.soap.ve

[jira] Commented: (GERONIMO-3124) Reminder: Switch Axis2 stack back to axis2-saaj from Sun SAAJ Impl

2007-05-21 Thread Jarek Gawor (JIRA)

[ 
https://issues.apache.org/jira/browse/GERONIMO-3124?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12497663
 ] 

Jarek Gawor commented on GERONIMO-3124:
---

No. Sun SAAJ implementation is the default implementation and right now that 
dependency is needed there. This should have no effect on Axis2 as Axis2 SAAJ 
implementation is preferred if it is found in the classpath.


> Reminder:  Switch Axis2 stack back to axis2-saaj from Sun SAAJ Impl
> ---
>
> Key: GERONIMO-3124
> URL: https://issues.apache.org/jira/browse/GERONIMO-3124
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>  Components: webservices
>Affects Versions: 2.0-M6
>Reporter: Donald Woods
> Assigned To: Lin Sun
>Priority: Critical
> Fix For: 2.0-M6
>
> Attachments: G3124.patch
>
>
> This is a placeholder, so we don't forget to rollback the temporary changes 
> made to use the Sun SAAJ Impl instead of the axis2-saaj jar.
> Most of the following changes will have to be rolled back - Revisions 530234, 
> 531260
> Author: dims
> Date: Wed Apr 18 19:18:50 2007
> New Revision: 530234
> URL: http://svn.apache.org/viewvc?view=rev&rev=530234
> Log:
> Switch Axis2 over to sun's saaj jars as well. Once we stabilize, we can 
> switch back to see what breaks with Axis2's saaj impl jars
> Modified:
> geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml
> geronimo/server/trunk/configs/axis2/pom.xml
> 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
> Modified: geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml
> URL: 
> http://svn.apache.org/viewvc/geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml?view=diff&rev=530234&r1=530233&r2=530234
> ==
> --- geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml (original)
> +++ geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml Wed Apr 18 
> 19:18:50 2007
> @@ -83,19 +83,11 @@
>  
>  
>  org.apache.axis2
> -axis2-saaj
> Modified: geronimo/server/trunk/configs/axis2/pom.xml
> URL: 
> http://svn.apache.org/viewvc/geronimo/server/trunk/configs/axis2/pom.xml?view=diff&rev=530234&r1=530233&r2=530234
> ==
> --- geronimo/server/trunk/configs/axis2/pom.xml (original)
> +++ geronimo/server/trunk/configs/axis2/pom.xml Wed Apr 18 19:18:50 2007
> @@ -110,19 +110,11 @@
>  
>  
>  org.apache.axis2
> -axis2-saaj
> -
> Modified: 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
> URL: 
> http://svn.apache.org/viewvc/geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java?view=diff&rev=530234&r1=530233&r2=530234
> ==
> --- 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
>  (original)
> +++ 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
>  Wed Apr 18 19:18:50 2007
> @@ -32,12 +32,12 @@
>
> "org.apache.axis.soap.SOAPFactoryImpl",
>
> "org.apache.axis.soap.SOAPConnectionFactoryImpl",
>
> "org.apache.axis.soap.SAAJMetaFactoryImpl"));
> -SAAJ_FACTORIES.put(SAAJUniverse.Type.AXIS2.toString(), 
> -   
> createSAAJInfo("org.apache.axis2.saaj.MessageFactoryImpl", 
> -  
> "org.apache.axis2.saaj.SOAPFactoryImpl", 
> -  
> "org.apache.axis2.saaj.SOAPConnectionFactoryImpl", 
> -  
> "org.apache.axis2.saaj.SAAJMetaFactoryImpl"));
> -SAAJ_FACTORIES.put(SAAJUniverse.Type.SUN.toString(), 
> +SAAJ_FACTORIES.put(SAAJUniverse.Type.AXIS2.toString(),
> +   
> createSAAJInfo("com.sun.xml.messaging.saaj.soap.ver1_1.SOAPMessageFactory1_1Impl",
> +  
> "com.sun.xml.messaging.saaj.soap.ver1_1.SOAPFactory1_1Impl",
> +  
> "com.sun.xml.messaging.saaj.client.p2p.HttpSOAPConnectionFactory",
> +  
> "com.sun.xml.messaging.saaj.soap.SAAJMet

[jira] Commented: (GERONIMO-3124) Reminder: Switch Axis2 stack back to axis2-saaj from Sun SAAJ Impl

2007-05-21 Thread Ted Kirby (JIRA)

[ 
https://issues.apache.org/jira/browse/GERONIMO-3124?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12497640
 ] 

Ted Kirby commented on GERONIMO-3124:
-

You missed a spot.  saaj-impl is still listed as a dependency in 
configs/webservices-common, which means saaj-impl will still be used a lot.  
axis2-saaj must replace saaj-impl in configs/webservices-common/pom.xml

> Reminder:  Switch Axis2 stack back to axis2-saaj from Sun SAAJ Impl
> ---
>
> Key: GERONIMO-3124
> URL: https://issues.apache.org/jira/browse/GERONIMO-3124
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>  Components: webservices
>Affects Versions: 2.0-M6
>Reporter: Donald Woods
> Assigned To: Donald Woods
>Priority: Critical
> Fix For: 2.0-M6
>
> Attachments: G3124.patch
>
>
> This is a placeholder, so we don't forget to rollback the temporary changes 
> made to use the Sun SAAJ Impl instead of the axis2-saaj jar.
> Most of the following changes will have to be rolled back - Revisions 530234, 
> 531260
> Author: dims
> Date: Wed Apr 18 19:18:50 2007
> New Revision: 530234
> URL: http://svn.apache.org/viewvc?view=rev&rev=530234
> Log:
> Switch Axis2 over to sun's saaj jars as well. Once we stabilize, we can 
> switch back to see what breaks with Axis2's saaj impl jars
> Modified:
> geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml
> geronimo/server/trunk/configs/axis2/pom.xml
> 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
> Modified: geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml
> URL: 
> http://svn.apache.org/viewvc/geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml?view=diff&rev=530234&r1=530233&r2=530234
> ==
> --- geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml (original)
> +++ geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml Wed Apr 18 
> 19:18:50 2007
> @@ -83,19 +83,11 @@
>  
>  
>  org.apache.axis2
> -axis2-saaj
> Modified: geronimo/server/trunk/configs/axis2/pom.xml
> URL: 
> http://svn.apache.org/viewvc/geronimo/server/trunk/configs/axis2/pom.xml?view=diff&rev=530234&r1=530233&r2=530234
> ==
> --- geronimo/server/trunk/configs/axis2/pom.xml (original)
> +++ geronimo/server/trunk/configs/axis2/pom.xml Wed Apr 18 19:18:50 2007
> @@ -110,19 +110,11 @@
>  
>  
>  org.apache.axis2
> -axis2-saaj
> -
> Modified: 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
> URL: 
> http://svn.apache.org/viewvc/geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java?view=diff&rev=530234&r1=530233&r2=530234
> ==
> --- 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
>  (original)
> +++ 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
>  Wed Apr 18 19:18:50 2007
> @@ -32,12 +32,12 @@
>
> "org.apache.axis.soap.SOAPFactoryImpl",
>
> "org.apache.axis.soap.SOAPConnectionFactoryImpl",
>
> "org.apache.axis.soap.SAAJMetaFactoryImpl"));
> -SAAJ_FACTORIES.put(SAAJUniverse.Type.AXIS2.toString(), 
> -   
> createSAAJInfo("org.apache.axis2.saaj.MessageFactoryImpl", 
> -  
> "org.apache.axis2.saaj.SOAPFactoryImpl", 
> -  
> "org.apache.axis2.saaj.SOAPConnectionFactoryImpl", 
> -  
> "org.apache.axis2.saaj.SAAJMetaFactoryImpl"));
> -SAAJ_FACTORIES.put(SAAJUniverse.Type.SUN.toString(), 
> +SAAJ_FACTORIES.put(SAAJUniverse.Type.AXIS2.toString(),
> +   
> createSAAJInfo("com.sun.xml.messaging.saaj.soap.ver1_1.SOAPMessageFactory1_1Impl",
> +  
> "com.sun.xml.messaging.saaj.soap.ver1_1.SOAPFactory1_1Impl",
> +  
> "com.sun.xml.messaging.saaj.client.p2p.HttpSOAPConnectionFactory",
> +  
> "com.sun.xml.messaging.saaj.soap.SAAJMetaF

[jira] Commented: (GERONIMO-3124) Reminder: Switch Axis2 stack back to axis2-saaj from Sun SAAJ Impl

2007-05-16 Thread Lin Sun (JIRA)

[ 
https://issues.apache.org/jira/browse/GERONIMO-3124?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12496319
 ] 

Lin Sun commented on GERONIMO-3124:
---

I also verified that this didn't break CXF by running the saaj-war test with 
the jetty assembly.

> Reminder:  Switch Axis2 stack back to axis2-saaj from Sun SAAJ Impl
> ---
>
> Key: GERONIMO-3124
> URL: https://issues.apache.org/jira/browse/GERONIMO-3124
> Project: Geronimo
>  Issue Type: Bug
>  Security Level: public(Regular issues) 
>  Components: webservices
>Affects Versions: 2.0-M6
>Reporter: Donald Woods
> Assigned To: Donald Woods
>Priority: Critical
> Fix For: 2.0-M6
>
> Attachments: G3124.patch
>
>
> This is a placeholder, so we don't forget to rollback the temporary changes 
> made to use the Sun SAAJ Impl instead of the axis2-saaj jar.
> Most of the following changes will have to be rolled back - Revisions 530234, 
> 531260
> Author: dims
> Date: Wed Apr 18 19:18:50 2007
> New Revision: 530234
> URL: http://svn.apache.org/viewvc?view=rev&rev=530234
> Log:
> Switch Axis2 over to sun's saaj jars as well. Once we stabilize, we can 
> switch back to see what breaks with Axis2's saaj impl jars
> Modified:
> geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml
> geronimo/server/trunk/configs/axis2/pom.xml
> 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
> Modified: geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml
> URL: 
> http://svn.apache.org/viewvc/geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml?view=diff&rev=530234&r1=530233&r2=530234
> ==
> --- geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml (original)
> +++ geronimo/server/trunk/configs/axis2-deployer/src/plan/plan.xml Wed Apr 18 
> 19:18:50 2007
> @@ -83,19 +83,11 @@
>  
>  
>  org.apache.axis2
> -axis2-saaj
> Modified: geronimo/server/trunk/configs/axis2/pom.xml
> URL: 
> http://svn.apache.org/viewvc/geronimo/server/trunk/configs/axis2/pom.xml?view=diff&rev=530234&r1=530233&r2=530234
> ==
> --- geronimo/server/trunk/configs/axis2/pom.xml (original)
> +++ geronimo/server/trunk/configs/axis2/pom.xml Wed Apr 18 19:18:50 2007
> @@ -110,19 +110,11 @@
>  
>  
>  org.apache.axis2
> -axis2-saaj
> -
> Modified: 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
> URL: 
> http://svn.apache.org/viewvc/geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java?view=diff&rev=530234&r1=530233&r2=530234
> ==
> --- 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
>  (original)
> +++ 
> geronimo/server/trunk/modules/geronimo-webservices/src/main/java/org/apache/geronimo/webservices/saaj/SAAJFactoryFinder.java
>  Wed Apr 18 19:18:50 2007
> @@ -32,12 +32,12 @@
>
> "org.apache.axis.soap.SOAPFactoryImpl",
>
> "org.apache.axis.soap.SOAPConnectionFactoryImpl",
>
> "org.apache.axis.soap.SAAJMetaFactoryImpl"));
> -SAAJ_FACTORIES.put(SAAJUniverse.Type.AXIS2.toString(), 
> -   
> createSAAJInfo("org.apache.axis2.saaj.MessageFactoryImpl", 
> -  
> "org.apache.axis2.saaj.SOAPFactoryImpl", 
> -  
> "org.apache.axis2.saaj.SOAPConnectionFactoryImpl", 
> -  
> "org.apache.axis2.saaj.SAAJMetaFactoryImpl"));
> -SAAJ_FACTORIES.put(SAAJUniverse.Type.SUN.toString(), 
> +SAAJ_FACTORIES.put(SAAJUniverse.Type.AXIS2.toString(),
> +   
> createSAAJInfo("com.sun.xml.messaging.saaj.soap.ver1_1.SOAPMessageFactory1_1Impl",
> +  
> "com.sun.xml.messaging.saaj.soap.ver1_1.SOAPFactory1_1Impl",
> +  
> "com.sun.xml.messaging.saaj.client.p2p.HttpSOAPConnectionFactory",
> +  
> "com.sun.xml.messaging.saaj.soap.SAAJMetaFactoryImpl"));
> +SAAJ_FACTORIES.put(SAAJUniverse.Type.SUN.toString(),
> 
> createSAAJ