Skip to content

IGNITE-18358 IGN-TX-5 on concurrent transactional single key load#1563

Merged
vldpyatkov merged 5 commits intoapache:mainfrom
gridgain:ignite-18358
Jan 27, 2023
Merged

IGNITE-18358 IGN-TX-5 on concurrent transactional single key load#1563
vldpyatkov merged 5 commits intoapache:mainfrom
gridgain:ignite-18358

Conversation

@vldpyatkov
Copy link
Copy Markdown
Contributor

Comment on lines 268 to 270

/** Failed to release a lock on a key. */
public static final int RELEASE_LOCK_ERR = TX_ERR_GROUP.registerErrorCode(4);

/** Failed to acquire a lock on a key due to a conflict. */
public static final int ACQUIRE_LOCK_ERR = TX_ERR_GROUP.registerErrorCode(5);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shouldnt we descrease the numbers of ACQUIRE_LOCK_ERR and following codes?

import static java.util.concurrent.CompletableFuture.completedFuture;
import static java.util.concurrent.CompletableFuture.failedFuture;
import static org.apache.ignite.internal.util.ExceptionUtils.withCause;
import static org.apache.ignite.lang.ErrorGroups.Common.UNKNOWN_ERR;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why UNKNOWN_ERR is deprecated? im not sure that it is good pattern to use deprecated value, but if it is ok, the annotation should be removed

@vldpyatkov vldpyatkov merged commit d93bc6f into apache:main Jan 27, 2023
@vldpyatkov vldpyatkov deleted the ignite-18358 branch January 27, 2023 09:34
lowka pushed a commit to gridgain/apache-ignite-3 that referenced this pull request Mar 18, 2023
lowka pushed a commit to gridgain/apache-ignite-3 that referenced this pull request Apr 19, 2023
isapego pushed a commit to isapego/ignite-3 that referenced this pull request Dec 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants