glassfish
  1. glassfish
  2. GLASSFISH-3623

PK Generation doesn't work with HSQL database

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Affects Version/s: 9.1peur1
    • Fix Version/s: not determined
    • Component/s: entity-persistence
    • Labels:
      None
    • Environment:

      Operating System: Windows XP
      Platform: All

    • Issuezilla Id:
      3,623
    • Status Whiteboard:
      Hide

      as91ur1-na as911-na

      Show
      as91ur1-na as911-na

      Description

      For some reason, no matter what configuration I use for Id generation I get an
      exception when using a HSQL DB:

      Exception Description: SEQ_GEN_SEQUENCE: platform HSQLPlatform doesn't support
      NativeSequence.

      It's the exact same exception with IDENTITY and SEQUENCE. (Which is strange, as
      really it should at least LOOK different with Identity).

      HSQL supports Sequence, and has done so since HSQL 1.7 (I believe).

      In big projects HSQL is a popular DB to use for unit tests, as it doesn't
      require much set up. It means that it isn't a prduction issue, but is seriously
      slows down the developers efficiency.

        Activity

        Hide
        magott added a comment -

        I just attached a zip with a possible solution to the problem. Its two different
        DatabasePlatform-implementations that can be used EITHER to support sequence or
        to support identity. I haven't tried having it all in one class. I'm not
        convinced this will work, it seems to me that Identity and Sequence is mutually
        exclusive the way it's been implemented in TopLink Essential.

        Feel free to edit the code, refactor the package name or whatever. It WOULD be
        nice to get a mention in the @author-tag in the javadoc though

        Show
        magott added a comment - I just attached a zip with a possible solution to the problem. Its two different DatabasePlatform-implementations that can be used EITHER to support sequence or to support identity. I haven't tried having it all in one class. I'm not convinced this will work, it seems to me that Identity and Sequence is mutually exclusive the way it's been implemented in TopLink Essential. Feel free to edit the code, refactor the package name or whatever. It WOULD be nice to get a mention in the @author-tag in the javadoc though
        Hide
        tware added a comment -

        Updated target milestone

        Show
        tware added a comment - Updated target milestone
        Hide
        Mitesh Meswani added a comment -

        Will not be fixed for V2.1

        Show
        Mitesh Meswani added a comment - Will not be fixed for V2.1
        Hide
        sanandal added a comment -

        "Reclassifying as P4 because this issue is not deemed "must fix" for this v2.1
        release whose primary release driver is SailFin.
        This issue will be scrubbed after this release and will be given the right
        priority for the next release."

        Show
        sanandal added a comment - "Reclassifying as P4 because this issue is not deemed "must fix" for this v2.1 release whose primary release driver is SailFin. This issue will be scrubbed after this release and will be given the right priority for the next release."
        Hide
        Tom Mueller added a comment -

        Bulk change to set fix version to "not determined" where the issue is open but the value is for a released version.

        Show
        Tom Mueller added a comment - Bulk change to set fix version to "not determined" where the issue is open but the value is for a released version.

          People

          • Assignee:
            tware
            Reporter:
            magott
          • Votes:
            1 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated: