Hi Anthony, It happened while the server is starting and creating a cache (while fills in the content of a cache based on the creation object's state). The NPE occurs when the "create region" command is executed before pdxRegistry is initialized. There is that part of the code where pdxRegistry is initialized: https://github.com/Nordix/geode/blob/develop/geode-core/src/main/java/org/apache/geode/internal/cache/xmlcache/CacheCreation.java#L529
Before this part of the code is executed it has that pdxRegistry is null, and it throws the NPE in findDiskStore. BR, Mario ________________________________ Šalje: Anthony Baker <bak...@vmware.com> Poslano: 7. srpnja 2021. 17:58 Prima: dev@geode.apache.org <dev@geode.apache.org> Predmet: Re: NullPointerException while create region during server restart When the NPE occurs, has the server completed its bootstrapping from cluster configuration yet? Anthony > On Jul 6, 2021, at 12:06 AM, Mario Kevo <mario.k...@est.tech> wrote: > > Hi Geode devs, > > I opened a new ticket https://issues.apache.org/jira/browse/GEODE-9409 > regarding NullPointerException on creating region while one of the servers is > restarting. > If we run the "create region" command through gfsh while the server is > starting it passed, but if the server is restarted then it fails. The > difference is that when we restarted the server, we kill them and start > again. As it has already a server directory, it takes more time to get the > server up as expected. > In that case, if we run the "create region" command it can happen that the > cache is not fully created and we are trying to do something on that. That > can lead to the NullPointerException, as creating region catches pdxRegistry > from the cache while doing findDiskStore, but sometimes it is not initialized > in the cache yet. So every method run against that will throw > NullPoniterException. > There is a part of the code where the exception is thrown: > > DiskStoreImpl findDiskStore(RegionAttributes regionAttributes, > InternalRegionArguments internalRegionArgs) { > // validate that persistent type registry is persistent > if (getAttributes().getDataPolicy().withPersistence()) { > getCache().getPdxRegistry().creatingPersistentRegion(); > } > > As I already mention, getPdxRegistry(LocalRegion.java) will be null if it is > not yet initialized in create(CacheCreation.java): > > DiskStoreAttributesCreation pdxRegDSC = initializePdxDiskStore(cache); > > cache.initializePdxRegistry(); > > createDiskStores(cache, pdxRegDSC); > > I tried to do some fixes, but without a success. 🙁 > It can be passed if we add some retry and sleep, but that is not acceptable. > > So if someone has some idea how to do some wait until pdxRegistry is > initialized or something else what will help us to avoid this problem? > > BR, > Mario
