glassfish
  1. glassfish
  2. GLASSFISH-263

Use a properties file for persistence provider specific properties

    Details

    • Type: Improvement Improvement
    • Status: Reopened
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 9.0pe
    • Fix Version/s: future release
    • Component/s: entity-persistence
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      263

      Description

      Have a property file which look like this:
      <provider-class-name1>.<provider-property-A>=<valueA>
      <provider-class-name1>.<provider-property-B>=<valueB>
      <provider-class-name2>.<provider-property-C>=<valueC>
      <provider-class-name2>.<provider-property-D>=<valueD>

      This can be loaded in PersistenceUnitLoaderImpl and then we can just pass the
      properties specific to a provider in the map paremeter of
      createContainerEntityManagerFactory().

      Thanks,
      Sahoo

        Activity

        Hide
        djclarke added a comment -

        I am unsure of the value of this enhancement. Currently the persistence.xml
        provides properties where vendor specific details can be provided. Going forward
        as more advanced extensions are supported additional implementation specific
        files will be required. I don't see the need to add in yet another file.

        If there is some concrete usecases that would warrant this additional file
        please provide them and re-open.

        Show
        djclarke added a comment - I am unsure of the value of this enhancement. Currently the persistence.xml provides properties where vendor specific details can be provided. Going forward as more advanced extensions are supported additional implementation specific files will be required. I don't see the need to add in yet another file. If there is some concrete usecases that would warrant this additional file please provide them and re-open.
        Hide
        Mitesh Meswani added a comment -

        This enhancement was filled as a reminder to generalize handling of various
        persistence provider in Glassfish. Reopening.
        Please note that this is not an issue with Toplink

        Show
        Mitesh Meswani added a comment - This enhancement was filled as a reminder to generalize handling of various persistence provider in Glassfish. Reopening. Please note that this is not an issue with Toplink
        Hide
        Sanjeeb Sahoo added a comment -

        Assigning to jpa team for taking due action

        Show
        Sanjeeb Sahoo added a comment - Assigning to jpa team for taking due action

          People

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

            Dates

            • Created:
              Updated: