Derby
  1. Derby
  2. DERBY-1650

derbytools.jar needs org.apache.derby.iapi.reference.Attribute even though that class contains only interface Strings

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 10.2.1.6
    • Fix Version/s: None
    • Component/s: Tools
    • Urgency:
      Normal

      Description

      A recent checkin to remove the reference to org.apache.derby.iapi.reference.Attribute in the derbytools.jar build caused numerous tests to fail. This class contains only Strings from an interface, so the compiler should compile out all references to this class. The reason for needing this class at runtime should be tracked down and fixed.

        Activity

        Hide
        Knut Anders Hatlen added a comment -

        Triaged for 10.5.2.

        I believe the dependency comes from this code in impl/tools/ij/URLCheck:

        Class att = Attribute.class;
        //Use reflection to get the list of valid keys from the Attribute class.
        //The Attribute class is an interface and therefore all the field
        //for it are public.
        Field[] fields = att.getFields();
        for (int i = 0; i < fields.length; i++)

        { Field aField = (Field)fields[i]; props.addElement(aField.get(att)); }
        Show
        Knut Anders Hatlen added a comment - Triaged for 10.5.2. I believe the dependency comes from this code in impl/tools/ij/URLCheck: Class att = Attribute.class; //Use reflection to get the list of valid keys from the Attribute class. //The Attribute class is an interface and therefore all the field //for it are public. Field[] fields = att.getFields(); for (int i = 0; i < fields.length; i++) { Field aField = (Field)fields[i]; props.addElement(aField.get(att)); }
        Hide
        Rick Hillegas added a comment -

        Unknown release vehicle.

        Show
        Rick Hillegas added a comment - Unknown release vehicle.
        Hide
        Andrew McIntyre added a comment -

        Not an urgent issue, moving this out to 10.2.2.

        Show
        Andrew McIntyre added a comment - Not an urgent issue, moving this out to 10.2.2.

          People

          • Assignee:
            Unassigned
            Reporter:
            Andrew McIntyre
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:

              Development