You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@storm.apache.org by srdo <gi...@git.apache.org> on 2017/11/05 15:42:48 UTC

[GitHub] storm pull request #2402: STORM-2800: Use Maven JAXB api artifact instead of...

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 <sr...@apache.org>
Date:   2017-11-04T21:24:58Z

    STORM-2800: Use Maven JAXB api artifact instead of assuming it is available from the JDK

----


---

[GitHub] storm pull request #2402: STORM-2800: Use Maven JAXB api artifact instead of...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/storm/pull/2402


---