[crac] RFR: Fix ordering of invocation on Resources [v3]

Radim Vansa duke at openjdk.org
Tue May 2 06:40:45 UTC 2023


On Fri, 28 Apr 2023 11:51:19 GMT, Anton Kozlov <akozlov at openjdk.org> wrote:

>> Radim Vansa has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - More fine-grained synchronization
>>  - Rework context ordering (round 2)
>>    
>>    * call afterRestore even if beforeCheckpoint throws
>>    * registering resource in previous/running context does not trigger exception immediatelly
>>    ** instead this will be one of the recorded exceptions and the resource has a chance to fire next time
>>    * we don't guarantee threads not deadlocking when trying to register a resource, though
>
> src/java.base/share/classes/jdk/crac/impl/PriorityContext.java line 21:
> 
>> 19: // CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
>> 20: // ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
>> 21: // POSSIBILITY OF SUCH DAMAGE.
> 
> Use standard copyright please

Uh, OK - I've copy pasted what's in the AbstractContextImpl as I've forked the class...

-------------

PR Review Comment: https://git.openjdk.org/crac/pull/60#discussion_r1182142187


More information about the crac-dev mailing list