Re: [PR] IGNITE-24658 Sql. Fix NPE in ReadWriteTransactionImpl#enlist [ignite-3]

2025-03-07 Thread via GitHub
korlov42 merged PR #5355: URL: https://github.com/apache/ignite-3/pull/5355 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@ig

Re: [PR] IGNITE-24658 Sql. Fix NPE in ReadWriteTransactionImpl#enlist [ignite-3]

2025-03-06 Thread via GitHub
vldpyatkov commented on code in PR #5355: URL: https://github.com/apache/ignite-3/pull/5355#discussion_r1983007793 ## modules/table/src/main/java/org/apache/ignite/internal/table/distributed/storage/InternalTableImpl.java: ## @@ -1997,7 +1997,7 @@ protected CompletableFuture en

Re: [PR] IGNITE-24658 Sql. Fix NPE in ReadWriteTransactionImpl#enlist [ignite-3]

2025-03-06 Thread via GitHub
korlov42 commented on code in PR #5355: URL: https://github.com/apache/ignite-3/pull/5355#discussion_r1983099460 ## modules/table/src/main/java/org/apache/ignite/internal/table/distributed/storage/InternalTableImpl.java: ## @@ -1997,7 +1997,7 @@ protected CompletableFuture enli

Re: [PR] IGNITE-24658 Sql. Fix NPE in ReadWriteTransactionImpl#enlist [ignite-3]

2025-03-06 Thread via GitHub
vldpyatkov commented on code in PR #5355: URL: https://github.com/apache/ignite-3/pull/5355#discussion_r1983007793 ## modules/table/src/main/java/org/apache/ignite/internal/table/distributed/storage/InternalTableImpl.java: ## @@ -1997,7 +1997,7 @@ protected CompletableFuture en

[PR] IGNITE-24658 Sql. Fix NPE in ReadWriteTransactionImpl#enlist [ignite-3]

2025-03-06 Thread via GitHub
korlov42 opened a new pull request, #5355: URL: https://github.com/apache/ignite-3/pull/5355 https://issues.apache.org/jira/browse/IGNITE-24658 The patch is split on 2 commits: first one adjust test framework (adds node eviction from physical topology) to highlight original issue (hig