jimma commented on a change in pull request #924:
URL: https://github.com/apache/cxf/pull/924#discussion_r827581610
##########
File path: core/src/main/java/org/apache/cxf/common/jaxb/JAXBContextCache.java
##########
@@ -336,7 +336,13 @@ private static JAXBContext createContext(final
Set<Class<?>> classes,
try {
ctx = AccessController.doPrivileged(new
PrivilegedExceptionAction<JAXBContext>() {
public JAXBContext run() throws Exception {
- return JAXBContext.newInstance(classes.toArray(new
Class<?>[0]), map);
+ //This is a workaround for CXF-8675
+ Class factoryClass =
ClassLoaderUtils.loadClass("org.glassfish.jaxb.runtime.v2.ContextFactory",
Review comment:
@reta You are right. I already moved this workaround in jaxb/JAXBUtils,
and other places will call in this new added `createContext` method. Please
review.
BTW, this jaxb-api issue is already fixed in upstream
https://github.com/eclipse-ee4j/jaxb-api/pull/225. But it will take some time
to have a 3.x release.
##########
File path: core/src/main/java/org/apache/cxf/common/jaxb/JAXBContextCache.java
##########
@@ -336,7 +336,13 @@ private static JAXBContext createContext(final
Set<Class<?>> classes,
try {
ctx = AccessController.doPrivileged(new
PrivilegedExceptionAction<JAXBContext>() {
public JAXBContext run() throws Exception {
- return JAXBContext.newInstance(classes.toArray(new
Class<?>[0]), map);
+ //This is a workaround for CXF-8675
+ Class factoryClass =
ClassLoaderUtils.loadClass("org.glassfish.jaxb.runtime.v2.ContextFactory",
Review comment:
@reta You are right. I already moved this workaround to jaxb/JAXBUtils,
and other places will call in this new added `createContext` method. Please
review.
BTW, this jaxb-api issue is already fixed in upstream
https://github.com/eclipse-ee4j/jaxb-api/pull/225. But it will take some time
to have a 3.x release.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
