Hi Dave,

Nice to here that you have no objections…
…now we hope that no one else has any objections ;o)

Best Regards,
Michael

> Am 13.10.2015 um 16:00 schrieb Vilaverde, David <dvilave...@informatica.com>:
> 
> Hi Michael,
> 
> There are no objections. I'll look for your commit later this week.
> 
> Thanks
> 
> Dave
> 
> ________________________________
> From: Bolz, Michael <michael.b...@sap.com>
> Sent: Tuesday, October 13, 2015 4:35:48 AM
> To: dev@olingo.apache.org
> Subject: [lazy consensus][DISCUSS] aalto-xml dependency
> 
> Hi Dave,
> 
> I don’t think that there would be a problem using woodstox.
> At lease with Olingo V2 replacing “woodstox” with “aalto-xml” worked without 
> known issues.
> But to be sure I would recommend to test it extensive before use it in any 
> productive environment.
> 
> However, IMHO the suggested change (in 
> https://issues.apache.org/jira/browse/OLINGO-799)
> should be done in Olingo (master). So that an user has the possibility to 
> change the used "XML Input Factory".
> 
> If there are no objections about this change I would commit / merge it  at 
> end of this week.
> 
> Best Regards,
> Michael
> 
> On 09 Oct 2015, at 21:56, Vilaverde, David 
> <dvilave...@informatica.com<mailto:dvilave...@informatica.com>> wrote:
> 
> Hi Michael,
> 
> Do you think there would be a problem with using woodstox (wstx-asl) instead 
> of aalto-xml?  I have a situation where metro and olingo are both on the 
> classpath and aalto-xml on the classpath causes metro to truncate some text 
> nodes in the SOAP payload.  As soon as we remove aalto-xml from the classpath 
> metro behaves normally.
> 
> I guess the real question is there a requirement for aalto-xml or will any 
> stax library work?
> 
> In any case I'll put in a JIRA issue and provide a patch/pull request via 
> github.
> 
> Thanks
> 
> Dave
> 
> -----Original Message-----
> From: mibo [mailto:m...@apache.org]
> Sent: Friday, October 09, 2015 12:53 PM
> To: dev@olingo.apache.org<mailto:dev@olingo.apache.org>
> Subject: Re: [DISCUSS] aalto-xml dependency
> 
> Hi David,
> 
> IMHO the change should be safe as long as Olingo only have one 
> „XMLInputFactory“ in the dependencies/classpath (aalto-xml).
> So JIRA issue and the „one liner patch“ is welcome.
> 
> BTW, I tested also Olingo V2 with „com.fasterxml:aalto-xml:0.9.10“...
> System.setProperty("javax.xml.stream.XMLInputFactory", 
> "com.fasterxml.aalto.stax.InputFactoryImpl");
> …and all „core“ tests worked without a problem.
> 
> Best Regards,
> Michael
> 
> Am 09.10.2015 um 17:04 schrieb Vilaverde, David 
> <dvilave...@informatica.com<mailto:dvilave...@informatica.com>>:
> 
> We currently use both OData V2 and V4.
> 
> Is it safe to change the FACTORY from explicitly depending on aalto-xml to 
> using XMLInputFactory.newInstance()?
> 
> If so I will gladly create a JIRA and work on a patch to contribute back.
> 
> Thanks
> 
> David
> 
> -----Original Message-----
> From: mibo [mailto:m...@apache.org]
> Sent: Friday, October 09, 2015 10:01 AM
> To: dev@olingo.apache.org<mailto:dev@olingo.apache.org>
> Subject: Re: [DISCUSS] aalto-xml dependency
> 
> Hello Dave,
> 
> one question, do you use Olingo V2 or V4 or both?
> 
> For Olingo V4 there exists currently no way to change the used 
> XMLInputFactory in the „ODataXmlDeserializer"
> private static final XMLInputFactory FACTORY = new InputFactoryImpl();
> 
> For Olingo V2 you can set the „System Property“ to change the used 
> „XMLInputFactory/XMLOutputFactory“.
> See example below for Woodstox/Default JDK6.
> 
> Woodstox:
> System.setProperty("javax.xml.stream.XMLInputFactory", 
> "com.ctc.wstx.stax.WstxInputFactory");
> System.setProperty("javax.xml.stream.XMLOutputFactory", 
> "com.ctc.wstx.stax.WstxOutputFactory");
> Default JDK6:
> System.setProperty("javax.xml.stream.XMLInputFactory", 
> "com.sun.xml.internal.stream.XMLInputFactoryImpl");
> System.setProperty("javax.xml.stream.XMLOutputFactory", 
> "com.sun.xml.internal.stream.XMLOutputFactoryImpl");
> 
> Best Regards,
> Michael
> 
> Am 09.10.2015 um 15:29 schrieb Vilaverde, David 
> <dvilave...@informatica.com<mailto:dvilave...@informatica.com>>:
> 
> Hi,
> 
> I currently have a product in which I am using Metro and OLingo.  The Metro 
> stack depends on wstx-asl and olingo on aalto-xml and I'm running into issues 
> where the XMLInputFactory finds aalto-xml over wstx-asl which is causing 
> other serialization issues for Metro.  I tried removing aalto-xml from the 
> classpath but then I have issues with OData V4 XML because of a 
> NoClassDefFound exception when initializing ODataXmlDeserializer.  I noticed 
> the org.apache.olingo.odata2.core.commons.XmlHelper doesn't directly import 
> aalto but instead uses XMLInputFactory.newInstance(), so the question is can 
> all other usages of aalto be replaced with XMLInputFactory or 
> XMLOutputFactory, etc?
> 
> If so, I would gladly open a Jira issue and contribute a patch back to the 
> OLingo project via github.
> 
> Dave
> 
> 
> 
> 

Attachment: signature.asc
Description: Message signed with OpenPGP using GPGMail

Reply via email to