https://bz.apache.org/bugzilla/show_bug.cgi?id=59139

            Bug ID: 59139
           Summary: undeployOldVersions sorts alphabetically though
                    version numbers are normally numeric in part
           Product: Tomcat 8
           Version: trunk
          Hardware: All
                OS: All
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Catalina
          Assignee: dev@tomcat.apache.org
          Reporter: louis.bur...@gmail.com

The following function is supposed to identify old applications deployed using
parallel deployment:
org.apache.catalina.startup.HostConfig#checkUndeploy

The application names are qualified with a '##' and a version identifier. This
would often be expected to be numeric. However, comparison is done using a
TreeSet:

        // Need ordered set of names
        SortedSet<String> sortedAppNames = new TreeSet<>();

This will result in alphabet ordering. 'myApp##9' will be considered newer than
'myApp##10'. This does not seem sensible or intuitive to me.

This can be worked around by ensuring zero padding or otherwise guaranteeing
lexicographical ordering.

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to