gaussianrecurrence commented on pull request #715: URL: https://github.com/apache/geode-native/pull/715#issuecomment-760115798
> I wonder if anyone of this is actually necessary. I think this idea of clearing the PDX registry is a hold over from the old global world. Under that old model if you "closed" the cache it was never really closed, especially from the .NET layer, thus it never really cleared the the PDX registry, which was also global. Now the PDX registry is owned by Cache and Cache can actually be closed. If the Cache is closed the registry is destroyed. I really think we should just eliminate all this code. Sorry Jacob for the delay in reviewing this, I've just come back from my vacations. Thing is this change is part (or at least I though so) of a bigger issue. What we are seeing is that whenever a native client is connected to a cluster and such cluster is stopped and restarted with a backup restore point, several request are failing... we noticed that it was due to the client being aware of some PdxTypes which the cluster does not know about due to having loaded a restore point previous in time... So this change solve the issues, but I am conducting some tests as it's unclear whether or not that's the full solution. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
