Uploaded image for project: 'Solr'
  1. Solr
  2. SOLR-13335

Upgrade to velocity 2.0 and velocity-tools 3.0

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 8.1, 9.0
    • None
    • None

    Description

      As part of looking to remove old commons-lang in SOLR-9079, found that velocity depends on it. Upgrading Velocity would allow completely removing commons-lang

      Copy some detail from SOLR-9079 investigation:

      • solr/contrib/velocity/ivy.xml doesn't even reference commons-lang
      • velocity 1.7 was released - 2010-11-29
      • LUCENE-5249 from 2013 was the last time velocity was changed in lucene/ivy-versions.properties
      • velocity-tools 2.0 has an optional dependency on commons-lang
      • velocity 1.7 has a hard dependency on commons-lang.

      Upgrading velocity 1.7 -> 2.0

      So if we want to finish removing commons-lang, we need to upgrade velocity.

      Attachments

        1. SOLR-13335.patch
          52 kB
          Kevin Risden
        2. SOLR-13335.patch
          48 kB
          Kevin Risden
        3. SOLR-13335.patch
          49 kB
          Kevin Risden
        4. SOLR-13335.patch
          37 kB
          Kevin Risden
        5. SOLR-13335.patch
          33 kB
          Kevin Risden
        6. SOLR-13335.patch
          43 kB
          Kevin Risden

        Issue Links

          Activity

            People

              krisden Kevin Risden
              krisden Kevin Risden
              Votes:
              0 Vote for this issue
              Watchers:
              5 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