JDO
  1. JDO
  2. JDO-709

Standardize field/property converters

    Details

      Description

      This request is to standardize a user's ability to specify conversions of fields or properties of persistence-capable classes. Currently, this is left to vendor extensions.

      1. JDO-709-01.patch
        7 kB
        Matthew T. Adams

        Activity

        Hide
        Andy Jefferson added a comment -

        Majority of the patch was applied, with some changes
        1. Apache copyright added
        2. Omitted the package-level capabilities since we need to get something for 95% of the use-cases first (of which package is not). We can always extend to that in the future.
        3. Renamed "Converters" to "Converts" and added Converter annotation to allow for "autoApply" setting a default of whether a converter should default to being applied (following JPA converters).

        Show
        Andy Jefferson added a comment - Majority of the patch was applied, with some changes 1. Apache copyright added 2. Omitted the package-level capabilities since we need to get something for 95% of the use-cases first (of which package is not). We can always extend to that in the future. 3. Renamed "Converters" to "Converts" and added Converter annotation to allow for "autoApply" setting a default of whether a converter should default to being applied (following JPA converters).
        Hide
        Andy Jefferson added a comment -

        Patch looks fine to me.
        RI already supports the basic idea so will be trivial to enable this.
        Clearly we also need the XML updating, but that can come next.

        Show
        Andy Jefferson added a comment - Patch looks fine to me. RI already supports the basic idea so will be trivial to enable this. Clearly we also need the XML updating, but that can come next.
        Hide
        Matthew T. Adams added a comment -

        Initial thoughts. Including revision number in patch filename, as I expect we'll iterate on this.

        Show
        Matthew T. Adams added a comment - Initial thoughts. Including revision number in patch filename, as I expect we'll iterate on this.

          People

          • Assignee:
            Matthew T. Adams
            Reporter:
            Matthew T. Adams
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:

              Development