dspavlov commented on a change in pull request #5856: IGNITE-10974 Release page 
lock even if exception throws on beforeReleaseWrite
URL: https://github.com/apache/ignite/pull/5856#discussion_r249041300
 
 

 ##########
 File path: 
modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/pagemem/PageMemoryImpl.java
 ##########
 @@ -1543,24 +1543,32 @@ private void writeUnlockPage(
         if (markDirty)
 
 Review comment:
   we have pluggable changeTracker which can be used for injecting custom 
logic. Its signature also includes IgniteCheckedException. Should we expand try 
block wider?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to