Details

    • Type: New Feature New Feature
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.11.0
    • Component/s: None
    • Labels:
      None
    • Patch Info:
      Patch Available
    • Estimated Complexity:
      Unknown

      Description

      Contribution of Camel-ZeroMQ component.

      I've taken the liberty of adding documentation here:
      https://cwiki.apache.org/confluence/display/CAMEL/ZeroMQ

      90% test coverage.

      1. camel-zeromq.tar.gz
        20 kB
        stephen samuel
      2. camel-zeromq-inital.patch
        80 kB
        stephen samuel

        Activity

        stephen samuel created issue -
        stephen samuel made changes -
        Field Original Value New Value
        Attachment camel-zeromq-inital.patch [ 12543472 ]
        stephen samuel made changes -
        Description Contribution of Camel-ZeroMQ component.

        Contribution of Camel-ZeroMQ component.

        I've taken the liberty of adding documentation here:
        https://cwiki.apache.org/confluence/display/CAMEL/ZeroMQ

        90% test coverage.
        Hide
        Claus Ibsen added a comment -

        ZeroMQ is LGPL license and cannot be accepted at ASF.

        However it can be accepted at Camel Extra
        https://code.google.com/a/apache-extras.org/p/camel-extra/?redir=1

        Show
        Claus Ibsen added a comment - ZeroMQ is LGPL license and cannot be accepted at ASF. However it can be accepted at Camel Extra https://code.google.com/a/apache-extras.org/p/camel-extra/?redir=1
        Hide
        stephen samuel added a comment -

        It's not using ZeroMQ's JZMQ driver, it's using the Akka ZeroMQ driver which is apache 2 licensed.
        http://doc.akka.io/docs/akka/snapshot/project/licenses.html
        The actual component has no dependencies on any part of the Zeromq project.

        Show
        stephen samuel added a comment - It's not using ZeroMQ's JZMQ driver, it's using the Akka ZeroMQ driver which is apache 2 licensed. http://doc.akka.io/docs/akka/snapshot/project/licenses.html The actual component has no dependencies on any part of the Zeromq project.
        Hide
        Claus Ibsen added a comment -

        You would need to rework the patch to be applicable to the camel-extra project.

        See this guide about new components, step 2 is about the licensing
        http://camel.apache.org/add-new-component-guide.html

        Show
        Claus Ibsen added a comment - You would need to rework the patch to be applicable to the camel-extra project. See this guide about new components, step 2 is about the licensing http://camel.apache.org/add-new-component-guide.html
        Hide
        stephen samuel added a comment -

        I've also mentioned that in the documenation, because the akka driver is a little slower, that users can easily switch to the zeromq driver (in fact I could write that and put that bit in the camel-extra project?)

        Show
        stephen samuel added a comment - I've also mentioned that in the documenation, because the akka driver is a little slower, that users can easily switch to the zeromq driver (in fact I could write that and put that bit in the camel-extra project?)
        Hide
        stephen samuel added a comment -

        I was wondering if you had thought about my comments that the zeromq component does not depend on zeromq gpl or lgpl code. Do you think it might be ok to use ?

        Show
        stephen samuel added a comment - I was wondering if you had thought about my comments that the zeromq component does not depend on zeromq gpl or lgpl code. Do you think it might be ok to use ?
        Hide
        Claus Ibsen added a comment -

        And you are sue the akka driver do not use any other 3rd party JARs? All the JARs that are pulled in must be ASF licesing compliant

        Show
        Claus Ibsen added a comment - And you are sue the akka driver do not use any other 3rd party JARs? All the JARs that are pulled in must be ASF licesing compliant
        Hide
        stephen samuel added a comment -

        Akka uses the scala binding for zeromq, which is apache licensed. You can see that in the pom here on maven central:
        http://search.maven.org/#artifactdetails%7Corg.zeromq%7Czeromq-scala-binding_2.9.1%7C0.0.6%7Cjar

        It is the Java binding for zeromq (JZMQ) that is LGPL licensed which Akka doesn't use. This is the key point from the Akka documentation "The ZeroMQ module for Akka is written against an API introduced in JZMQ, which uses JNI to interact with the native ZeroMQ library. Instead of using JZMQ, the module uses ZeroMQ binding for Scala that uses the native ZeroMQ library through JNA. In other words, the only native library that this module requires is the native ZeroMQ library." http://doc.akka.io/docs/akka/snapshot/scala/zeromq.html

        The native library they refer to is the libzmq that is built when you compile zeromq from source (there is no binary distribution for linux). http://www.zeromq.org/intro:get-the-software

        Show
        stephen samuel added a comment - Akka uses the scala binding for zeromq, which is apache licensed. You can see that in the pom here on maven central: http://search.maven.org/#artifactdetails%7Corg.zeromq%7Czeromq-scala-binding_2.9.1%7C0.0.6%7Cjar It is the Java binding for zeromq (JZMQ) that is LGPL licensed which Akka doesn't use. This is the key point from the Akka documentation "The ZeroMQ module for Akka is written against an API introduced in JZMQ, which uses JNI to interact with the native ZeroMQ library. Instead of using JZMQ, the module uses ZeroMQ binding for Scala that uses the native ZeroMQ library through JNA. In other words, the only native library that this module requires is the native ZeroMQ library." http://doc.akka.io/docs/akka/snapshot/scala/zeromq.html The native library they refer to is the libzmq that is built when you compile zeromq from source (there is no binary distribution for linux). http://www.zeromq.org/intro:get-the-software
        Hide
        Claus Ibsen added a comment -

        Well if its okay for Akka then its most likely okay for ASF. That said though I would like a few more from the Camel team to take a look as well.

        Show
        Claus Ibsen added a comment - Well if its okay for Akka then its most likely okay for ASF. That said though I would like a few more from the Camel team to take a look as well.
        Hide
        stephen samuel added a comment -

        In the meantime is there anything else I can add / improve for this patch ?

        Show
        stephen samuel added a comment - In the meantime is there anything else I can add / improve for this patch ?
        Hide
        Claus Ibsen added a comment -
        Show
        Claus Ibsen added a comment - Posted on @dev to get feedback on this contribution http://camel.465427.n5.nabble.com/ZeroMQ-contribution-Any-thoughts-on-licensing-hack-tp5719948.html
        Hide
        James Carman added a comment -

        What's wrong with just leaving this in "extras" for now while the licensing issues are investigated?

        Show
        James Carman added a comment - What's wrong with just leaving this in "extras" for now while the licensing issues are investigated?
        Hide
        stephen samuel added a comment -

        I put a reply on. I think using term hack makes it sound like its bad

        Show
        stephen samuel added a comment - I put a reply on. I think using term hack makes it sound like its bad
        Claus Ibsen made changes -
        Fix Version/s Future [ 12315692 ]
        Fix Version/s 2.11.0 [ 12321695 ]
        Hide
        Claus Ibsen added a comment -

        Lets host this at Camel Extra as discussed on the dev forum

        Show
        Claus Ibsen added a comment - Lets host this at Camel Extra as discussed on the dev forum
        Claus Ibsen made changes -
        Fix Version/s 2.11.0 [ 12321695 ]
        Fix Version/s Future [ 12315692 ]
        Priority Minor [ 4 ] Major [ 3 ]
        Hide
        Claus Ibsen added a comment -

        Stephen would it be possible to attach a .zip/tar with the component. As we need to move it to camel-extra. So it may be easier to add if its not a patch.

        An alternative would be for you to migrate the code to camel-extra and attach a new patch.
        Mind that the code at Camel extra will be a different license; if you are okay with that.

        Show
        Claus Ibsen added a comment - Stephen would it be possible to attach a .zip/tar with the component. As we need to move it to camel-extra. So it may be easier to add if its not a patch. An alternative would be for you to migrate the code to camel-extra and attach a new patch. Mind that the code at Camel extra will be a different license; if you are okay with that.
        Hide
        stephen samuel added a comment -

        Ok I've added the src and pom as a tar. Is that acceptable?

        Show
        stephen samuel added a comment - Ok I've added the src and pom as a tar. Is that acceptable?
        stephen samuel made changes -
        Attachment camel-zeromq.tar.gz [ 12548165 ]
        Claus Ibsen made changes -
        Assignee Claus Ibsen [ davsclaus ]
        Hide
        Claus Ibsen added a comment -

        The component is now in the camel extra source code, and will be part of next release.

        To have it work with OSGi that's another case whether that is possible. But that's a task on the Camel Extra side, to work on OSGi supports of the components in this project.

        Show
        Claus Ibsen added a comment - The component is now in the camel extra source code, and will be part of next release. To have it work with OSGi that's another case whether that is possible. But that's a task on the Camel Extra side, to work on OSGi supports of the components in this project.
        Claus Ibsen made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        70d 4h 55m 1 Claus Ibsen 11/Nov/12 16:42

          People

          • Assignee:
            Claus Ibsen
            Reporter:
            stephen samuel
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development