Uploaded image for project: 'Cassandra'
  1. Cassandra
  2. CASSANDRA-15647

Mismatching dependencies between cassandra dist and cassandra-all pom

    XMLWordPrintableJSON

    Details

    • Bug Category:
      Code - Bug - Unclear Impact
    • Severity:
      Low
    • Complexity:
      Low Hanging Fruit
    • Discovered By:
      Code Inspection
    • Platform:
      All
    • Impacts:
      None
    • Since Version:
    • Test and Documentation Plan:
      Hide

       

      ant write-poms

      {{ mvn dependency:tree f build/apache-cassandra*-SNAPSHOT.pom -Dverbose -Dincludes=net.java.dev.jna}}

      Show
        ant write-poms {{ mvn dependency:tree f build/apache-cassandra *-SNAPSHOT.pom -Dverbose -Dincludes=net.java.dev.jna}}

      Description

      I noticed that the cassandra distribution (tar.gz) dependencies doesn't match the dependency list for the cassandra-all that is available at maven central.

      Cassandra distribution only includes jna 4.2.2.
      But, the maven dependency also include jna-platform 4.4.0

      Breakdown of relevant maven dependencies:

      ```
      [INFO] +- org.apache.cassandra:cassandra-all:jar:4.0-alpha3:provided
      [INFO] | +- net.java.dev.jna:jna:jar:4.2.2:provided
      [INFO] | +- net.openhft:chronicle-threads:jar:1.16.0:provided
      [INFO] | | - net.openhft:affinity:jar:3.1.7:provided
      [INFO] | | - net.java.dev.jna:jna-platform:jar:4.4.0:provided
      ```

      As you can see, jna is a direct dependency and jna-platform is a transitive dependency from chronicle-threads.

      I expected this issue to had been fixed by https://github.com/apache/cassandra/pull/240/, but this change seem to have being reverted, as no longer in trunk.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                rssvihla Ryan Svihla
                Reporter:
                velobr Marvin Froeder
                Authors:
                Ryan Svihla
                Reviewers:
                Michael Semb Wever
              • Votes:
                0 Vote for this issue
                Watchers:
                6 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: