glassfish
  1. glassfish
  2. GLASSFISH-17477

different vmcid and minor codes between GF 2.1 and GF 3.1.1

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Affects Version/s: 3.1.1
    • Fix Version/s: None
    • Component/s: orb
    • Labels:
      None

      Description

      We noticed getting different vmcid and minor codes between GF 2.1 and GF
      3.1.1.

      For example, in case of a connection failure due to a non-existing
      destination server:

      • GF 2.1.x:org.omg.CORBA.COMM_FAILURE: vmcid: SUN minor code: 201
        completed: No
      • GF 3.1.1:org.omg.CORBA.COMM_FAILURE: vmcid: OMG minor code: 1
        completed: No

      In case of a communications timeout while waiting for a response:

      • GF 2.1.x:org.omg.CORBA.COMM_FAILURE: vmcid: SUN minor code: 219
        completed: Maybe
      • GF 3.1.1:org.omg.CORBA.COMM_FAILURE: vmcid: OMG minor code: 19
        completed: Maybe

      The following excerpt from the code seems to indicate it's a bug:

      src\share\classes\com\sun\corba\se\spi\orbutil\logex\corba\CorbaExtension.java

      final int base = orbex.omgException() ?
      SUNVMCID.value :
      OMGVMCID.value ;

      (it looks like the two values were placed the wrong way around)

      About the minor codes, they are calculated in the same class using "200*orbex.group() + log.id()".
      Shouldn't the group indices start from 1, as below?

      public static final int ORBUtilGroup = 1 ;
      public static final int ActivationGroup = 2 ;
      public static final int NamingGroup = 3 ;
      public static final int InterceptorsGroup = 4 ;
      public static final int POAGroup = 5 ;
      public static final int IORGroup = 6 ;
      public static final int UtilGroup = 7 ;

        Activity

        There are no comments yet on this issue.

          People

          • Assignee:
            Harshad Vilekar
            Reporter:
            Dies Koper
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated: