geode-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Bruce Schuchardt <>
Subject Re: Proposal: GEODE-4367 - Return PDXInstance when Domain Object can't be found
Date Thu, 25 Jan 2018 15:38:31 GMT

I've found the current read-serialized property to be pretty useless.

Having said that, what if the value isn't actually in serialized form in 
the local cache?  Is Geode supposed to serialize it & return it?  What 
if it isn't PDX-serialized?  Do we return a byte array?

On 1/24/18 12:21 PM, Dan Smith wrote:
> Is this really just a workaround for the fact that the read-serialized flag
> applies to the whole cache? I can see that if you have mix of regions with
> and without domain classes on the server you might want this feature. Can
> you provide some more background on your use case?
> IMO we should get rid of read-serialized in favor of APIs that let the user
> decide whether they get a domain class or a PdxInstance.
> -Dan
> On Wed, Jan 24, 2018 at 9:58 AM, Galen O'Sullivan <>
> wrote:
>> Hi Addison,
>> What kind of setup do you have that is causing you to have PDX serialized
>> objects that cannot be deserialized? Do you have classes that are present
>> on some servers but not others?
>> This change would break the contract of region.get() , which is that it
>> returns a value of a type that can be put into the region.
>> Returning something that isn't what the user is expecting to be in the
>> region would require users to check for PdxInstances every time they get a
>> value -- even though the type annotations say that you can't get a
>> PdxInstance back (except for Region<Object,Object> ).
>> I think it would be better to have a second API that allows users to get
>> and put PdxInstance objects in regions. That way, if they want to handle
>> the exceptional case where they have a serialized object that can't be
>> deserialized, they can catch the ClassNotFound exception and get the
>> underlying PdxInstance.
>> I do think that the possibility of a ClassNotFoundException should be
>> documented in the Region API.
>> Cheers,
>> Galen
>> On Tue, Jan 23, 2018 at 2:56 PM, Addison Huddy <> wrote:
>>> Hi Geode Devs,
>>> I'm proposing the following change to how we handle deserialization when
>>> Domain Objects can't be found and pdx-serialize=false.
>>> Looking forward to the discussion.
>>> \ah

View raw message