[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