GitHub user srdo opened a pull request:
https://github.com/apache/storm/pull/2402
STORM-2800: Use Maven JAXB api artifact instead of assuming it is avaâ¦
â¦ilable from the JDK
See https://issues.apache.org/jira/browse/STORM-2800.
jaxb-api is dual licensed under CDDL 1.1 and GPL. CDDL is a category B
license, which the FAQ at https://www.apache.org/legal/resolved.html#category-b
says should be labeled somewhere users will read it. Do we have a document like
that already, or where do I put the notice?
The uses of JAXB are:
* Uses of DataTypeConverter in storm-client and storm-autocreds. jaxb-api
includes a default implementation of this converter
(https://github.com/javaee/jaxb-spec/blob/master/jaxb-api/src/main/java/javax/xml/bind/DatatypeConverterImpl.java),
so we don't need to include a full JAXB implementation to use this.
* Throws declaration of JAXBException in storm-pmml, which is thrown from a
class in one of the org.jpmml libraries. The org.jpmml:pmml-model library
includes a JAXB implementation, so we shouldn't need to add it ourselves.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/srdo/storm STORM-2800
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/storm/pull/2402.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #2402
----
commit c3b56624356224743f5e22a186807a3875ddd83e
Author: Stig Rohde Døssing <[email protected]>
Date: 2017-11-04T21:24:58Z
STORM-2800: Use Maven JAXB api artifact instead of assuming it is available
from the JDK
----
---
