OpenJPA
  1. OpenJPA
  2. OPENJPA-2087

Remove or limit the use of the ConcreteClassGenerator

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.2.0
    • Component/s: build / infrastructure
    • Labels:
      None
    • Patch Info:
      Patch Available

      Description

      Now that we have to only support Java 6 in OpenJPA 2.x, there is discussion about revisiting the issue of removing or limiting the use of the ConcreteClassGenerator. This was originally discussed in OpenJPA-1520 [1]. This would help cleanup some of the code and would hopefully result in some performance gains, despite that it didn't seem to help with earlier primitive tests.

      [1] https://issues.apache.org/jira/browse/OPENJPA-1520

      1. OPENJPA-2087-1.patch
        86 kB
        Jacob Nowosatka
      2. OPENJPA-2087-2.patch
        12 kB
        Jacob Nowosatka

        Issue Links

          Activity

          Hide
          Jacob Nowosatka added a comment -

          The patch removes most uses of the ConcreteClassGenerator, with a few exceptions.

          Show
          Jacob Nowosatka added a comment - The patch removes most uses of the ConcreteClassGenerator, with a few exceptions.
          Hide
          Jacob Nowosatka added a comment -

          The patch has been updated to fix a few minor cosmetic issues.

          Show
          Jacob Nowosatka added a comment - The patch has been updated to fix a few minor cosmetic issues.
          Hide
          Kevin Sutter added a comment -

          In order to be compilable with Java 7, a couple of additional "not supported" methods have to be introduced into our classes.

          Show
          Kevin Sutter added a comment - In order to be compilable with Java 7, a couple of additional "not supported" methods have to be introduced into our classes.
          Hide
          Jacob Nowosatka added a comment -

          The classes have been updated so that they implement the required methods according to the Java 7 spec.

          Show
          Jacob Nowosatka added a comment - The classes have been updated so that they implement the required methods according to the Java 7 spec.
          Hide
          Albert Lee added a comment -

          Close issue in preparation for 2.2.0 release.

          Show
          Albert Lee added a comment - Close issue in preparation for 2.2.0 release.

            People

            • Assignee:
              Kevin Sutter
              Reporter:
              Jacob Nowosatka
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development