[ 
https://issues.apache.org/jira/browse/IGNITE-25020?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17940748#comment-17940748
 ] 
Roman Puchkovskiy commented on IGNITE-25020:
--------------------------------------------

The patch looks good to me

> TableManagerTest throws NullPointerException when the colocation feature is 
> enabled
> -----------------------------------------------------------------------------------
>
>                 Key: IGNITE-25020
>                 URL: https://issues.apache.org/jira/browse/IGNITE-25020
>             Project: Ignite
>          Issue Type: Bug
>            Reporter: Vyacheslav Koptilin
>            Assignee: Vyacheslav Koptilin
>            Priority: Major
>              Labels: ignite-3
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> TableManagerTest.testPreconfiguredTable throws the following exception:
> {noformat}
> [2025-04-03T18:42:10,430][ERROR][%node1%metastorage-watch-executor-2][FailureManager]
>  Critical system error detected. Will be handled accordingly to configured 
> handler [hnd=NoOpFailureHandler [super=AbstractFailureHandler 
> [ignoredFailureTypes=UnmodifiableSet [SYSTEM_WORKER_BLOCKED, 
> SYSTEM_CRITICAL_OPERATION_TIMEOUT]]], failureCtx=CRITICAL_ERROR]
>  org.apache.ignite.internal.lang.IgniteInternalException: Failed to apply 
> catalog update.
>     at 
> org.apache.ignite.internal.catalog.CatalogManagerImpl$OnUpdateHandlerImpl.lambda$handle$1(CatalogManagerImpl.java:506)
>  ~[main/:?]
>     at 
> java.base/java.util.concurrent.CompletableFuture.uniWhenComplete(CompletableFuture.java:859)
>  ~[?:?]
>     at 
> java.base/java.util.concurrent.CompletableFuture$UniWhenComplete.tryFire(CompletableFuture.java:837)
>  ~[?:?]
>     at 
> java.base/java.util.concurrent.CompletableFuture.postComplete(CompletableFuture.java:506)
>  ~[?:?]
>     at 
> java.base/java.util.concurrent.CompletableFuture.complete(CompletableFuture.java:2075)
>  ~[?:?]
>     at 
> org.apache.ignite.internal.util.CompletableFutures.lambda$copyStateTo$2(CompletableFutures.java:160)
>  ~[main/:?]
>     at 
> java.base/java.util.concurrent.CompletableFuture.uniWhenComplete(CompletableFuture.java:859)
>  ~[?:?]
>     at 
> java.base/java.util.concurrent.CompletableFuture.uniWhenCompleteStage(CompletableFuture.java:883)
>  ~[?:?]
>     at 
> java.base/java.util.concurrent.CompletableFuture.whenComplete(CompletableFuture.java:2253)
>  ~[?:?]
>     at 
> org.apache.ignite.internal.causality.BaseVersionedValue.copyState(BaseVersionedValue.java:335)
>  ~[main/:?]
>     at 
> org.apache.ignite.internal.causality.BaseVersionedValue.complete(BaseVersionedValue.java:219)
>  ~[main/:?]
>     at 
> org.apache.ignite.internal.causality.IncrementalVersionedValue.lambda$completeInternal$2(IncrementalVersionedValue.java:283)
>  ~[main/:?]
>     at 
> java.base/java.util.concurrent.CompletableFuture.uniWhenComplete(CompletableFuture.java:859)
>  ~[?:?]
>     at 
> java.base/java.util.concurrent.CompletableFuture.uniWhenCompleteStage(CompletableFuture.java:883)
>  ~[?:?]
>     at 
> java.base/java.util.concurrent.CompletableFuture.whenComplete(CompletableFuture.java:2253)
>  ~[?:?]
>     at 
> org.apache.ignite.internal.causality.IncrementalVersionedValue.completeInternal(IncrementalVersionedValue.java:283)
>  ~[main/:?]
>     at 
> org.apache.ignite.internal.causality.IncrementalVersionedValue$1.onUpdate(IncrementalVersionedValue.java:112)
>  ~[main/:?]
>     at 
> org.apache.ignite.internal.metastorage.server.WatchProcessor.notifyUpdateRevisionListeners(WatchProcessor.java:377)
>  ~[main/:?]
>     at 
> org.apache.ignite.internal.metastorage.server.WatchProcessor.lambda$notifyWatches$3(WatchProcessor.java:196)
>  ~[main/:?]
>     at 
> java.base/java.util.concurrent.CompletableFuture$UniCompose.tryFire(CompletableFuture.java:1072)
>  [?:?]
>     at 
> java.base/java.util.concurrent.CompletableFuture$Completion.run(CompletableFuture.java:478)
>  [?:?]
>     at 
> java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
>  [?:?]
>     at 
> java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
>  [?:?]
>     at java.base/java.lang.Thread.run(Thread.java:834) [?:?]
> Caused by: java.util.concurrent.CompletionException: 
> java.lang.NullPointerException
>     at 
> java.base/java.util.concurrent.CompletableFuture.encodeThrowable(CompletableFuture.java:314)
>  ~[?:?]
>     at 
> java.base/java.util.concurrent.CompletableFuture.completeThrowable(CompletableFuture.java:319)
>  ~[?:?]
>     at 
> java.base/java.util.concurrent.CompletableFuture.uniHandle(CompletableFuture.java:932)
>  ~[?:?]
>     at 
> java.base/java.util.concurrent.CompletableFuture.uniHandleStage(CompletableFuture.java:946)
>  ~[?:?]
>     at 
> java.base/java.util.concurrent.CompletableFuture.handle(CompletableFuture.java:2268)
>  ~[?:?]
>     at 
> org.apache.ignite.internal.causality.IncrementalVersionedValue.update(IncrementalVersionedValue.java:225)
>  ~[main/:?]
>     at 
> org.apache.ignite.internal.table.distributed.TableManager.prepareTableResourcesAndLoadHavingZoneReadLock(TableManager.java:803)
>  ~[main/:?]
>     at 
> org.apache.ignite.internal.table.distributed.TableManager.prepareTableResourcesAndLoadToZoneReplica(TableManager.java:784)
>  ~[main/:?]
>     at 
> org.apache.ignite.internal.table.distributed.TableManager.prepareTableResourcesAndLoadToZoneReplica(TableManager.java:757)
>  ~[main/:?]
>     at 
> org.apache.ignite.internal.event.AbstractEventProducer.fireEvent(AbstractEventProducer.java:88)
>  ~[main/:?]
>     at 
> org.apache.ignite.internal.catalog.CatalogManagerImpl.access$000(CatalogManagerImpl.java:76)
>  ~[main/:?]
>     at 
> org.apache.ignite.internal.catalog.CatalogManagerImpl$OnUpdateHandlerImpl.handle(CatalogManagerImpl.java:491)
>  ~[main/:?]
>     at 
> org.apache.ignite.internal.catalog.CatalogManagerImpl$OnUpdateHandlerImpl.handle(CatalogManagerImpl.java:458)
>  ~[main/:?]
>     at 
> org.apache.ignite.internal.catalog.storage.UpdateLogImpl$UpdateListener.onUpdate(UpdateLogImpl.java:339)
>  ~[main/:?]
>     at 
> org.apache.ignite.internal.metastorage.server.Watch.onUpdate(Watch.java:67) 
> ~[main/:?]
>     at 
> org.apache.ignite.internal.metastorage.server.WatchProcessor.notifyWatches(WatchProcessor.java:231)
>  ~[main/:?]
>     at 
> org.apache.ignite.internal.metastorage.server.WatchProcessor.lambda$notifyWatches$3(WatchProcessor.java:193)
>  ~[main/:?]
>     ... 5 more
>     Suppressed: java.lang.NullPointerException
>         at 
> org.apache.ignite.internal.table.distributed.TableManager.unlockZoneForRead(TableManager.java:855)
>  ~[main/:?]
>         at 
> org.apache.ignite.internal.table.distributed.TableManager.lambda$prepareTableResourcesAndLoadToZoneReplica$29(TableManager.java:785)
>  ~[main/:?]
>         at 
> java.base/java.util.concurrent.CompletableFuture.uniWhenComplete(CompletableFuture.java:859)
>  ~[?:?]
>         at 
> java.base/java.util.concurrent.CompletableFuture$UniWhenComplete.tryFire(CompletableFuture.java:837)
>  ~[?:?]
>         at 
> java.base/java.util.concurrent.CompletableFuture.postComplete(CompletableFuture.java:506)
>  ~[?:?]
>         at 
> java.base/java.util.concurrent.CompletableFuture.complete(CompletableFuture.java:2075)
>  ~[?:?]
>         at 
> org.apache.ignite.internal.util.CompletableFutures.lambda$copyStateTo$2(CompletableFutures.java:160)
>  ~[main/:?]
>         at 
> java.base/java.util.concurrent.CompletableFuture.uniWhenComplete(CompletableFuture.java:859)
>  ~[?:?]
>         at 
> java.base/java.util.concurrent.CompletableFuture.uniWhenCompleteStage(CompletableFuture.java:883)
>  ~[?:?]
>         at 
> java.base/java.util.concurrent.CompletableFuture.whenComplete(CompletableFuture.java:2253)
>  ~[?:?]
>         at 
> org.apache.ignite.internal.causality.BaseVersionedValue.copyState(BaseVersionedValue.java:335)
>  ~[main/:?]
>         at 
> org.apache.ignite.internal.causality.BaseVersionedValue.complete(BaseVersionedValue.java:219)
>  ~[main/:?]
>         at 
> org.apache.ignite.internal.causality.IncrementalVersionedValue.lambda$completeInternal$2(IncrementalVersionedValue.java:283)
>  ~[main/:?]
>         at 
> java.base/java.util.concurrent.CompletableFuture.uniWhenComplete(CompletableFuture.java:859)
>  ~[?:?]
>         at 
> java.base/java.util.concurrent.CompletableFuture.uniWhenCompleteStage(CompletableFuture.java:883)
>  ~[?:?]
>         at 
> java.base/java.util.concurrent.CompletableFuture.whenComplete(CompletableFuture.java:2253)
>  ~[?:?]
>         at 
> org.apache.ignite.internal.causality.IncrementalVersionedValue.completeInternal(IncrementalVersionedValue.java:283)
>  ~[main/:?]
>         at 
> org.apache.ignite.internal.causality.IncrementalVersionedValue$1.onUpdate(IncrementalVersionedValue.java:112)
>  ~[main/:?]
>         at 
> org.apache.ignite.internal.metastorage.server.WatchProcessor.notifyUpdateRevisionListeners(WatchProcessor.java:377)
>  ~[main/:?]
>         at 
> org.apache.ignite.internal.metastorage.server.WatchProcessor.lambda$notifyWatches$3(WatchProcessor.java:196)
>  ~[main/:?]
>         at 
> java.base/java.util.concurrent.CompletableFuture$UniCompose.tryFire(CompletableFuture.java:1072)
>  [?:?]
>         at 
> java.base/java.util.concurrent.CompletableFuture$Completion.run(CompletableFuture.java:478)
>  [?:?]
>         at 
> java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
>  [?:?]
>         at 
> java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
>  [?:?]
>         at java.base/java.lang.Thread.run(Thread.java:834) [?:?]
> Caused by: java.lang.NullPointerException
>     at 
> org.apache.ignite.internal.table.distributed.TableManager.lambda$prepareTableResourcesAndLoadHavingZoneReadLock$32(TableManager.java:804)
>  ~[main/:?]
>     at 
> org.apache.ignite.internal.util.IgniteUtils.inBusyLock(IgniteUtils.java:868) 
> ~[main/:?]
>     at 
> org.apache.ignite.internal.table.distributed.TableManager.lambda$prepareTableResourcesAndLoadHavingZoneReadLock$33(TableManager.java:804)
>  ~[main/:?]
>     at 
> java.base/java.util.concurrent.CompletableFuture.uniHandle(CompletableFuture.java:930)
>  ~[?:?]
>     at 
> java.base/java.util.concurrent.CompletableFuture.uniHandleStage(CompletableFuture.java:946)
>  ~[?:?]
>     at 
> java.base/java.util.concurrent.CompletableFuture.handle(CompletableFuture.java:2268)
>  ~[?:?]
>     at 
> org.apache.ignite.internal.causality.IncrementalVersionedValue.update(IncrementalVersionedValue.java:225)
>  ~[main/:?]
>     at 
> org.apache.ignite.internal.table.distributed.TableManager.prepareTableResourcesAndLoadHavingZoneReadLock(TableManager.java:803)
>  ~[main/:?]
>     at 
> org.apache.ignite.internal.table.distributed.TableManager.prepareTableResourcesAndLoadToZoneReplica(TableManager.java:784)
>  ~[main/:?]
>     at 
> org.apache.ignite.internal.table.distributed.TableManager.prepareTableResourcesAndLoadToZoneReplica(TableManager.java:757)
>  ~[main/:?]
>     at 
> org.apache.ignite.internal.event.AbstractEventProducer.fireEvent(AbstractEventProducer.java:88)
>  ~[main/:?]
>     at 
> org.apache.ignite.internal.catalog.CatalogManagerImpl.access$000(CatalogManagerImpl.java:76)
>  ~[main/:?]
>     at 
> org.apache.ignite.internal.catalog.CatalogManagerImpl$OnUpdateHandlerImpl.handle(CatalogManagerImpl.java:491)
>  ~[main/:?]
>     at 
> org.apache.ignite.internal.catalog.CatalogManagerImpl$OnUpdateHandlerImpl.handle(CatalogManagerImpl.java:458)
>  ~[main/:?]
>     at 
> org.apache.ignite.internal.catalog.storage.UpdateLogImpl$UpdateListener.onUpdate(UpdateLogImpl.java:339)
>  ~[main/:?]
>     at 
> org.apache.ignite.internal.metastorage.server.Watch.onUpdate(Watch.java:67) 
> ~[main/:?]
>     at 
> org.apache.ignite.internal.metastorage.server.WatchProcessor.notifyWatches(WatchProcessor.java:231)
>  ~[main/:?]
>     at 
> org.apache.ignite.internal.metastorage.server.WatchProcessor.lambda$notifyWatches$3(WatchProcessor.java:193)
>  ~[main/:?]
>     ... 5 more
> {noformat}
> Looks like we need to adjust the mock of `PartitionReplicaLifecycleManager`



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

Reply via email to