Re: [jira] [Commented] (GEODE-2441) Remove PDXAutoSerializer

2017-03-01 Thread David Kimura
It has been moved into a contrib directory outside the core sources where
other standalone tools can live too.

https://github.com/apache/geode-native/tree/develop/contrib/pdxautoserializer

Thanks,
David

On Wed, Mar 1, 2017 at 9:28 AM, Michael Stolz  wrote:

> We should make sure to find a new home for the PDXAutoSerializer, not just
> remove it.
> It is an important part of the product.
> Without it, PDX can't claim C++ compatibility.
> I assume a separate Jira has bin opened for that work?
>
>
> --
> Mike Stolz
> Principal Engineer, GemFire Product Manager
> Mobile: +1-631-835-4771
>
> On Wed, Mar 1, 2017 at 10:41 AM, ASF GitHub Bot (JIRA) 
> wrote:
>
> >
> > [ https://issues.apache.org/jira/browse/GEODE-2441?page=
> > com.atlassian.jira.plugin.system.issuetabpanels:comment-
> > tabpanel=15890406#comment-15890406 ]
> >
> > ASF GitHub Bot commented on GEODE-2441:
> > ---
> >
> > Github user echobravopapa commented on a diff in the pull request:
> >
> > https://github.com/apache/geode-native/pull/42#discussion_r103713640
> >
> > --- Diff: src/quickstart/cpp/CMakeLists.txt ---
> > @@ -26,16 +26,6 @@ elseif(UNIX)
> >  set(DYNAMIC_LIBRARY_PATH LD_LIBRARY_PATH=${
> NATIVECLIENT_DIR}/lib)
> >  endif()
> >
> > --- End diff --
> >
> > -verbose flag LTGM, I had made the same changes locally
> >
> >
> > > Remove PDXAutoSerializer
> > > -
> > >
> > > Key: GEODE-2441
> > > URL: https://issues.apache.org/jira/browse/GEODE-2441
> > > Project: Geode
> > >  Issue Type: Bug
> > >  Components: native client
> > >Reporter: Ernest Burghardt
> > >
> > > Remove PDXAutoSerializer utility that generates PDX serialization C++
> > source that you can include in your project to (de)serialize your C++
> > classes.
> >
> >
> >
> > --
> > This message was sent by Atlassian JIRA
> > (v6.3.15#6346)
> >
>


Re: [jira] [Commented] (GEODE-2441) Remove PDXAutoSerializer

2017-03-01 Thread Michael Stolz
We should make sure to find a new home for the PDXAutoSerializer, not just
remove it.
It is an important part of the product.
Without it, PDX can't claim C++ compatibility.
I assume a separate Jira has bin opened for that work?


--
Mike Stolz
Principal Engineer, GemFire Product Manager
Mobile: +1-631-835-4771

On Wed, Mar 1, 2017 at 10:41 AM, ASF GitHub Bot (JIRA) 
wrote:

>
> [ https://issues.apache.org/jira/browse/GEODE-2441?page=
> com.atlassian.jira.plugin.system.issuetabpanels:comment-
> tabpanel=15890406#comment-15890406 ]
>
> ASF GitHub Bot commented on GEODE-2441:
> ---
>
> Github user echobravopapa commented on a diff in the pull request:
>
> https://github.com/apache/geode-native/pull/42#discussion_r103713640
>
> --- Diff: src/quickstart/cpp/CMakeLists.txt ---
> @@ -26,16 +26,6 @@ elseif(UNIX)
>  set(DYNAMIC_LIBRARY_PATH LD_LIBRARY_PATH=${NATIVECLIENT_DIR}/lib)
>  endif()
>
> --- End diff --
>
> -verbose flag LTGM, I had made the same changes locally
>
>
> > Remove PDXAutoSerializer
> > -
> >
> > Key: GEODE-2441
> > URL: https://issues.apache.org/jira/browse/GEODE-2441
> > Project: Geode
> >  Issue Type: Bug
> >  Components: native client
> >Reporter: Ernest Burghardt
> >
> > Remove PDXAutoSerializer utility that generates PDX serialization C++
> source that you can include in your project to (de)serialize your C++
> classes.
>
>
>
> --
> This message was sent by Atlassian JIRA
> (v6.3.15#6346)
>