You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@ignite.apache.org by "Roman Puchkovskiy (Jira)" <ji...@apache.org> on 2024/07/19 10:13:00 UTC

[jira] [Commented] (IGNITE-22699) Anti-hijack protection of containsAllAsync() is broken

    [ https://issues.apache.org/jira/browse/IGNITE-22699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17867273#comment-17867273 ] 

Roman Puchkovskiy commented on IGNITE-22699:
--------------------------------------------

The patch looks good to me

> Anti-hijack protection of containsAllAsync() is broken
> ------------------------------------------------------
>
>                 Key: IGNITE-22699
>                 URL: https://issues.apache.org/jira/browse/IGNITE-22699
>             Project: Ignite
>          Issue Type: Bug
>            Reporter: Roman Puchkovskiy
>            Assignee: Mikhail Efremov
>            Priority: Major
>              Labels: ignite-3
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> Both KeyValueView and RecordView have containsAllAsync(). Its implementations in PublicApiThreadingKeyValueView and PublicApiThreadingRecordView are broken because they treat the operation as a sync one, so no anti-hijack protection is added.
> Also, the new operation should be added to {{ItKvRecordApiThreadingTest.KeyValueViewAsyncOperation}} and {{ItKvRecordApiThreadingTest.RecordViewAsyncOperation}} to cover the anti-hijack protection for the new methods.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)