Details

    • Type: Task Task
    • Status: Resolved
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.7
    • Component/s: contrib
    • Labels:
      None

      Description

      To reduce the overhead of maintaining the NOTICE and LICENSE files, we need to mavenize most checked-in jars. Out of the current checked-in jars, the following ones cannot be mavenized -

      1. lib/sbt-launch.jar: This is required to fire up the build system of Kafka (SBT)
      2. core/lib/zkclient-20110412.jar: Kafka uses a patched zkclient jar
      3. contrib/hadoop-consumer/lib/piggybank.jar: This is not available through Maven
      4. contrib/hadoop-consumer/lib/pig-0.8.0-core.jar: This is also not available through Maven

      1. kafka-222.patch
        7 kB
        Neha Narkhede
      2. rm_jars.sh
        0.6 kB
        Jakob Homan

        Issue Links

          Activity

          Neha Narkhede created issue -
          Neha Narkhede made changes -
          Field Original Value New Value
          Status Open [ 1 ] In Progress [ 3 ]
          Neha Narkhede made changes -
          Status In Progress [ 3 ] Patch Available [ 10002 ]
          Hide
          Neha Narkhede added a comment -

          This patch mavenizes most jars, except the ones mentioned in the description of the jira

          Show
          Neha Narkhede added a comment - This patch mavenizes most jars, except the ones mentioned in the description of the jira
          Neha Narkhede made changes -
          Attachment kafka-222.patch [ 12506084 ]
          Hide
          Jay Kreps added a comment -

          Don't we need to do the same thing now on Josh's patch? The longer we hold off on that patch the more it will drift.

          Show
          Jay Kreps added a comment - Don't we need to do the same thing now on Josh's patch? The longer we hold off on that patch the more it will drift.
          Hide
          Neha Narkhede added a comment -

          Josh's patch has been reviewed. We found that it wasn't created off the latest trunk and has a bug.

          Show
          Neha Narkhede added a comment - Josh's patch has been reviewed. We found that it wasn't created off the latest trunk and has a bug.
          Jakob Homan made changes -
          Link This issue blocks KAFKA-221 [ KAFKA-221 ]
          Hide
          Jakob Homan added a comment -

          +1. Verified can still test and build after this. BTW, here's a script to rm away the newly deleted jars from git, since patch application doesn't do that.

          Show
          Jakob Homan added a comment - +1. Verified can still test and build after this. BTW, here's a script to rm away the newly deleted jars from git, since patch application doesn't do that.
          Jakob Homan made changes -
          Attachment rm_jars.sh [ 12506172 ]
          Hide
          Jun Rao added a comment -

          +1

          Show
          Jun Rao added a comment - +1
          Neha Narkhede made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]

            People

            • Assignee:
              Neha Narkhede
              Reporter:
              Neha Narkhede
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development