Hi Scott & Emilio, I agree that getXMLDocBuilder should be synchronized. I've committed your change.
Thanks, -Matt > -----Original Message----- > From: Scott Nichol [mailto:[EMAIL PROTECTED]] > Sent: Friday, May 17, 2002 1:03 PM > To: [EMAIL PROTECTED] > Subject: [PATCH] Re: is SOAP thread-safe? > > > I don't know about whether DocumentBuilderFactory is not thread-safe, but > refreshDocumentBuilderFactory and getXMLDocBuilder need to be > synchronized with > respect to one another, as the former assigns to the dbf member then calls > methods on it to set parsing attributes. The assignment and > attribute setting > should be atomic, right? > > I've attached the trivial patch that changes getXMLDocBuilder to be > synchronized. > > Scott > > ----- Original Message ----- > From: "Emilio Belmonte" <[EMAIL PROTECTED]> > To: <[EMAIL PROTECTED]> > Sent: Thursday, May 16, 2002 3:09 PM > Subject: is SOAP thread-safe? > > > > I noticed that this method: > > > > XmlParserUtils.getXMLDocBuilder() > > > > is not synchronized. According to the JAXP documentation, the > > DocumentBuilderFactory class is not thread-safe. Is there any > reason for > > not synchronizing it? > > > > Thank you, > > > > --Emilio > > >