[ 
http://issues.apache.org/jira/browse/DERBY-2107?page=comments#action_12456903 ] 
            
Daniel John Debrunner commented on DERBY-2107:
----------------------------------------------

Sorry, I was wrong on issue 2), got confused mentally walking through the code.

Though the patch does remove this comment: for the situation in 2)

   // just deadlock out ...

I think some comment should exist there indicating that the code path is not 
expected or maybe even throw an exception there.
The patched code I think is confusing without some form of comment as to what 
is expected.

> Move page latching out of the lock manager
> ------------------------------------------
>
>                 Key: DERBY-2107
>                 URL: http://issues.apache.org/jira/browse/DERBY-2107
>             Project: Derby
>          Issue Type: Improvement
>          Components: Store, Services, Performance
>    Affects Versions: 10.3.0.0
>            Reporter: Knut Anders Hatlen
>         Assigned To: Knut Anders Hatlen
>            Priority: Minor
>         Attachments: derby-2107-1a.diff, derby-2107-1a.stat
>
>
> Latching of pages could be done more efficiently locally in store than in the 
> lock manager. See the discussion here: 
> http://thread.gmane.org/gmane.comp.apache.db.derby.devel/33135

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to