RE: [jira] [Commented] (GEODE-2722) ReflectionBasedAutoSerializer should be used by default

2017-03-27 Thread Real Wes 


“So the default pattern would match all class names?”

This is the most usual case and I believe should therefore be the default.  
Placing limitations is the exceptional case.  It’s rather frequent during 
development that I’ll change my package names and the serializer will blow up 
because I forgot to change the pattern. In the common case, it’s really a 
nuisance.







From: Darrel Schneider (JIRA)<mailto:j...@apache.org>
Sent: Monday, March 27, 2017 8:51 PM
To: dev@geode.apache.org<mailto:dev@geode.apache.org>
Subject: [jira] [Commented] (GEODE-2722) ReflectionBasedAutoSerializer should 
be used by default



[ 
https://issues.apache.org/jira/browse/GEODE-2722?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15944321#comment-15944321
 ]

Darrel Schneider commented on GEODE-2722:
-

So the default pattern would match all class names?

> ReflectionBasedAutoSerializer should be used by default
> ---
>
> Key: GEODE-2722
> URL: https://issues.apache.org/jira/browse/GEODE-2722
> Project: Geode
>  Issue Type: Improvement
>  Components: serialization
>Reporter: Swapnil Bawaskar
>
> We should not require the user to configure anything when inserting data in 
> Geode. ReflectionBasedAutoSerializer should be set by default on Cache
> startup (if one is not specified already).
> Also, the pattern required to configure ReflectionBasedAutoSerializer should 
> be made optional: Please see:
> <https://github.com/apache/geode/blob/8bf39571471642beaaa36c9626a61a90bd3803c2/geode-core/src/main/java/org/apache/geode/pdx/internal/AutoSerializableManager.java#L213>
> Please look at this thread for discussion on the dev list: 
> https://lists.apache.org/thread.html/c89974d08d7675d3a636872bce5e838d578df5759c5c1acae611d322@%3Cdev.geode.apache.org%3E



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[jira] [Commented] (GEODE-2722) ReflectionBasedAutoSerializer should be used by default

2017-03-27 Thread Darrel Schneider (JIRA) 

[ 
https://issues.apache.org/jira/browse/GEODE-2722?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15944333#comment-15944333
 ] 

Darrel Schneider commented on GEODE-2722:
-

The PdxSerializer is used before java serialization. So if you have a class 
that implements java serialization (by implementing Serializable or 
Externalizable or one of the other ways ObjectOutputStream supports) it would 
no longer be java serialized but would instead be serialized as a PDX. So 
making this change could break existing applications that rely on java 
serialization.
To get the old behaviour you should be able to configure the pdx-serializer to 
be null.


> ReflectionBasedAutoSerializer should be used by default
> ---
>
> Key: GEODE-2722
> URL: https://issues.apache.org/jira/browse/GEODE-2722
> Project: Geode
>  Issue Type: Improvement
>  Components: serialization
>Reporter: Swapnil Bawaskar
>
> We should not require the user to configure anything when inserting data in 
> Geode. ReflectionBasedAutoSerializer should be set by default on Cache
> startup (if one is not specified already). 
> Also, the pattern required to configure ReflectionBasedAutoSerializer should 
> be made optional: Please see:
> 
> Please look at this thread for discussion on the dev list: 
> https://lists.apache.org/thread.html/c89974d08d7675d3a636872bce5e838d578df5759c5c1acae611d322@%3Cdev.geode.apache.org%3E



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[jira] [Commented] (GEODE-2722) ReflectionBasedAutoSerializer should be used by default

2017-03-27 Thread Darrel Schneider (JIRA) 

[ 
https://issues.apache.org/jira/browse/GEODE-2722?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15944321#comment-15944321
 ] 

Darrel Schneider commented on GEODE-2722:
-

So the default pattern would match all class names?

> ReflectionBasedAutoSerializer should be used by default
> ---
>
> Key: GEODE-2722
> URL: https://issues.apache.org/jira/browse/GEODE-2722
> Project: Geode
>  Issue Type: Improvement
>  Components: serialization
>Reporter: Swapnil Bawaskar
>
> We should not require the user to configure anything when inserting data in 
> Geode. ReflectionBasedAutoSerializer should be set by default on Cache
> startup (if one is not specified already). 
> Also, the pattern required to configure ReflectionBasedAutoSerializer should 
> be made optional: Please see:
> 
> Please look at this thread for discussion on the dev list: 
> https://lists.apache.org/thread.html/c89974d08d7675d3a636872bce5e838d578df5759c5c1acae611d322@%3Cdev.geode.apache.org%3E



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)