OpenJPA
  1. OpenJPA
  2. OPENJPA-1296 WriteBehindCache Problems
  3. OPENJPA-1237

WriteBehind configuration exception not thrown when it should be

    Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: 2.1.0
    • Fix Version/s: None
    • Component/s: writebehind
    • Labels:
      None
    • Patch Info:
      Patch Available

      Description

      It seems like there should be exception(s) thrown when the new writebehind cache is not configured properly. For example, this is not a valid PU configuration since there is no DataCache specified.

      <persistence-unit name="WriteBehind_JSE_INVALID_PU1">
      <mapping-file>META-INF/orm.xml</mapping-file>
      <properties>
      <property name="openjpa.DataCache" value="false"/>
      <property name="openjpa.WriteBehindCache" value="true"/>
      <property name="openjpa.WriteBehindCallback" value="true(sleepTime=15000)"/>
      </properties>
      </persistence-unit>

      Yet there is no error or exception thrown when this statement is executed:

      emf = Persistence.createEntityManagerFactory("WriteBehind_JSE_INVALID_PU1");

        Activity

        Tim McConnell created issue -
        Tim McConnell made changes -
        Field Original Value New Value
        Patch Info [Patch Available]
        Hide
        Tim McConnell added a comment -

        This patch will force a WriteBehindConfigurationException when the WriteBehind Cache is configured without configuring the DataCache.

        Show
        Tim McConnell added a comment - This patch will force a WriteBehindConfigurationException when the WriteBehind Cache is configured without configuring the DataCache.
        Tim McConnell made changes -
        Attachment OPENJPA-1237.patch [ 12416164 ]
        Hide
        Tim McConnell added a comment -

        This second patch includes testcases to ensure that the appropriate WriteBehindConfigurationException is thrown for invalid configurations and an EntityManagerFactory is instantiated for valid configurations. Also, it will now properly throw a WriteBehindConfigurationException in all cases where required WriteBehind configuration parameters are omitted.

        Show
        Tim McConnell added a comment - This second patch includes testcases to ensure that the appropriate WriteBehindConfigurationException is thrown for invalid configurations and an EntityManagerFactory is instantiated for valid configurations. Also, it will now properly throw a WriteBehindConfigurationException in all cases where required WriteBehind configuration parameters are omitted.
        Tim McConnell made changes -
        Attachment OPENJPA-1237-2.patch [ 12416280 ]
        Tim McConnell made changes -
        Attachment OPENJPA-1237.patch [ 12416164 ]
        Hide
        Tim McConnell added a comment -

        This third patch will circumvent the adverse impact that these new configuration testcases are having on the other WriteBehind testcases in trunk. That problem will be addressed in another WriteBehind JIRA I'll open momentarily.

        Show
        Tim McConnell added a comment - This third patch will circumvent the adverse impact that these new configuration testcases are having on the other WriteBehind testcases in trunk. That problem will be addressed in another WriteBehind JIRA I'll open momentarily.
        Tim McConnell made changes -
        Attachment OPENJPA-1237-3.patch [ 12416650 ]
        Tim McConnell made changes -
        Attachment OPENJPA-1237-2.patch [ 12416280 ]
        Tim McConnell made changes -
        Component/s writebehind [ 12312973 ]
        Component/s kernel [ 12311302 ]
        Tim McConnell made changes -
        Parent OPENJPA-1296 [ 12435519 ]
        Issue Type Bug [ 1 ] Sub-task [ 7 ]
        Donald Woods made changes -
        Assignee Tim McConnell [ mcconne ] Donald Woods [ drwoods ]
        Donald Woods made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Hide
        Donald Woods added a comment -

        Committed by Mike on 8/15 in r804539.

        Show
        Donald Woods added a comment - Committed by Mike on 8/15 in r804539.
        Donald Woods made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Assignee Donald Woods [ drwoods ] Michael Dick [ mikedd ]
        Fix Version/s 2.0.0-M3 [ 12314148 ]
        Fix Version/s 2.0.0 [ 12314019 ]
        Resolution Fixed [ 1 ]
        Michael Dick made changes -
        Affects Version/s 2.1.0 [ 12314542 ]
        Affects Version/s 2.0.0 [ 12314019 ]
        Fix Version/s 2.1.0 [ 12314542 ]
        Fix Version/s 2.0.0-M3 [ 12314148 ]
        Hide
        Michael Dick added a comment -

        Reopening issues with WriteBehind.

        Show
        Michael Dick added a comment - Reopening issues with WriteBehind.
        Michael Dick made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Michael Dick made changes -
        Assignee Michael Dick [ mikedd ]
        Fix Version/s 2.1.0 [ 12314542 ]
        Michael Dick made changes -
        Status Reopened [ 4 ] Closed [ 6 ]
        Resolution Won't Fix [ 2 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open In Progress In Progress
        34d 7h 57m 1 Donald Woods 14/Sep/09 13:38
        In Progress In Progress Resolved Resolved
        11m 41s 1 Donald Woods 14/Sep/09 13:50
        Resolved Resolved Reopened Reopened
        130d 4h 1 Michael Dick 22/Jan/10 17:51
        Reopened Reopened Closed Closed
        242d 3h 27m 1 Michael Dick 21/Sep/10 21:18

          People

          • Assignee:
            Unassigned
            Reporter:
            Tim McConnell
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development