Uploaded image for project: 'Ignite'
  1. Ignite
  2. IGNITE-16852

Ignite compatibility framework can exclude unnecessary jars from test java process classpath

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 2.13
    • None

    Description

      Ignite compatibility framework can exclude unnecessary jars from test java process classpath.

      Compatibility framework excludes from calsspath classes which versions were explicitly specified. See IgniteCompatibilityAbstractTest#getProcessProxyJvmArgs.

      The exclusion mechanism is based on the specified dependency names
      See IgniteCompatibilityAbstractTest#getExcluded
      Dependency#sourcePathTemplate
      Dependency#artifactPathTemplate

      Since we use the following condition to filter dependency from classpath

        if (excluded.stream().noneMatch(path::contains))
      

      The following situation can occur:

      Assume that we have two dependencies with names
      `spring` and `spring-data-commons`. `spring` dependency version is specified explicitly and must be excluded. But since we check, if dependency jar path CONTAINS Dependency#artifactPathTemplate result both `spring` and `spring-data-commons` jars is excluded from the classpath.

      It can be fixed by attaching '/' to Dependency#artifactPathTemplate result.

      Attachments

        Issue Links

          Activity

            People

              mmuzaf Maxim Muzafarov
              PetrovMikhail Mikhail Petrov
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 20m
                  20m