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

2017-03-07 Thread ASF GitHub Bot (JIRA) 

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

ASF GitHub Bot commented on GEODE-2441:
---

Github user asfgit closed the pull request at:

https://github.com/apache/geode-native/pull/42


> 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)

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

2017-03-07 Thread ASF subversion and git services (JIRA) 

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

ASF subversion and git services commented on GEODE-2441:


Commit 7da9d90bdd28619193b044be9ab3a226c50ffd0c in geode-native's branch 
refs/heads/develop from [~dkimura]
[ https://git-wip-us.apache.org/repos/asf?p=geode-native.git;h=7da9d90 ]

GEODE-2441: Remove leftover pdx auto serializer references

This closes #42.


> 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)

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

2017-03-07 Thread ASF GitHub Bot (JIRA) 

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

ASF GitHub Bot commented on GEODE-2441:
---

Github user pivotal-jbarrett commented on the issue:

https://github.com/apache/geode-native/pull/42
  
@dgkimura You need to remove the C++ parser and Antlr from the LICENSE and 
dist/LICENSE files too.


> 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 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)
>

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

2017-03-01 Thread ASF GitHub Bot (JIRA) 

[ 
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)

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

2017-02-28 Thread ASF GitHub Bot (JIRA) 

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

ASF GitHub Bot commented on GEODE-2441:
---

Github user dgkimura commented on a diff in the pull request:

https://github.com/apache/geode-native/pull/42#discussion_r103572486
  
--- 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 --

Perhaps I'm missing context?  I'm not sure what it means.


> 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)

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

2017-02-28 Thread ASF GitHub Bot (JIRA) 

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

ASF GitHub Bot commented on GEODE-2441:
---

Github user dgkimura commented on a diff in the pull request:

https://github.com/apache/geode-native/pull/42#discussion_r103572155
  
--- Diff: src/quickstart/runcpp.sh.in ---
@@ -150,9 +149,6 @@ else
   "24")
 exname='PdxInstance'
   ;;
-  "25")
-exname='PdxAutoSerializer'
-  ;;
   "26")
--- End diff --

Done.


> 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)

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

2017-02-28 Thread ASF GitHub Bot (JIRA) 

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

ASF GitHub Bot commented on GEODE-2441:
---

Github user dgkimura commented on a diff in the pull request:

https://github.com/apache/geode-native/pull/42#discussion_r103572140
  
--- Diff: src/quickstart/runcpp.sh.in ---
@@ -69,7 +69,6 @@ else
 echo 22.PdxRemoteQuery
 echo 23.PdxSerializer
 echo 24.PdxInstance
-echo 25.PdxAutoSerializer
 echo 26.Quit
--- End diff --

Yes, thanks for catching.


> 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)

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

2017-02-28 Thread ASF GitHub Bot (JIRA) 

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

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_r103568625
  
--- Diff: src/quickstart/runcpp.sh.in ---
@@ -150,9 +149,6 @@ else
   "24")
 exname='PdxInstance'
   ;;
-  "25")
-exname='PdxAutoSerializer'
-  ;;
   "26")
--- End diff --

see above


> 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)

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

2017-02-28 Thread ASF GitHub Bot (JIRA) 

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

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_r103568564
  
--- Diff: src/quickstart/runcpp.sh.in ---
@@ -69,7 +69,6 @@ else
 echo 22.PdxRemoteQuery
 echo 23.PdxSerializer
 echo 24.PdxInstance
-echo 25.PdxAutoSerializer
 echo 26.Quit
--- End diff --

can we make Quit n-1?


> 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)

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

2017-02-28 Thread ASF GitHub Bot (JIRA) 

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

ASF GitHub Bot commented on GEODE-2441:
---

Github user dgkimura commented on a diff in the pull request:

https://github.com/apache/geode-native/pull/42#discussion_r103564551
  
--- Diff: src/tests/cli/NewFwkLib/NewFwkLib.csproj.in ---
@@ -272,9 +272,6 @@
 
   SmokePerf\NoopAuthInit.cs
 
-
-  PdxTest\PdxAutoSerializerObj.cs
--- End diff --

Hmm.. besides the mistake that this referenced file isn't in the 
pull-request, I'm not sure this can go so easily.


> 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)

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

2017-02-28 Thread ASF GitHub Bot (JIRA) 

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

ASF GitHub Bot commented on GEODE-2441:
---

GitHub user dgkimura opened a pull request:

https://github.com/apache/geode-native/pull/42

GEODE-2441: Remove leftover pdx auto serializer references

Remove leftover references from previous commits under GEODE-2441.  After 
this pull-request there are no references returned from grep in src directory.

`$ grep -n -R -i pdxautoserializer *`

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/dgkimura/geode-native feature/GEODE-2441

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/geode-native/pull/42.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #42






> 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)

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

2017-02-24 Thread Jacob S. Barrett (JIRA) 

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

Jacob S. Barrett commented on GEODE-2441:
-

I am still seeing PDX Auto Serializer code in the C++ client. 

- src/pdxautoserializer/ASCLIBuiltins.hpp
- src/cppcache/include/geode/PdxAutoSerializer.hpp
- src/quickstart/cpp/PdxAutoSerializer.cpp 

And referenced in:
- src/tests/cpp/testobject/CMakeLists.txt 
- src/quickstart/cpp/CMakeLists.txt 
- src/quickstart/runcpp.sh.in 
- src/quickstart/runcpp.bat.in 



> 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)

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

2017-02-24 Thread ASF subversion and git services (JIRA) 

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

ASF subversion and git services commented on GEODE-2441:


Commit aeba76554a7d17c6a0c553f42a0e44e7270d28b1 in geode-native's branch 
refs/heads/develop from [~dkimura]
[ https://git-wip-us.apache.org/repos/asf?p=geode-native.git;h=aeba765 ]

GEODE-2441: Remove antlr reference from core sources

This closes #33.


> 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)

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

2017-02-24 Thread ASF GitHub Bot (JIRA) 

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

ASF GitHub Bot commented on GEODE-2441:
---

Github user asfgit closed the pull request at:

https://github.com/apache/geode-native/pull/33


> 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)

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

2017-02-24 Thread ASF GitHub Bot (JIRA) 

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

ASF GitHub Bot commented on GEODE-2441:
---

Github user pivotal-jbarrett commented on the issue:

https://github.com/apache/geode-native/pull/33
  
@PivotalSarge CMakeList.txt.in is there to build Antlr with CMake rather 
than the very complicated and broken build process it ships with.


> 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)

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

2017-02-24 Thread ASF GitHub Bot (JIRA) 

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

ASF GitHub Bot commented on GEODE-2441:
---

Github user dgkimura commented on the issue:

https://github.com/apache/geode-native/pull/33
  
@PivotalSarge This is the relevant code after the patch is applied so my 
guess is there was a workaround in `antlr` that was probably non-portable.

```cpp
#if ( _MSC_VER == 1200 )
// VC6 seems to need this
// note that this is not a standard C++ include file.
# include 
#else
#include 
#include 
#endif
```


> 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)

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

2017-02-24 Thread ASF GitHub Bot (JIRA) 

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

ASF GitHub Bot commented on GEODE-2441:
---

GitHub user dgkimura opened a pull request:

https://github.com/apache/geode-native/pull/33

GEODE-2441: Remove antlr reference from core sources

References to antlr are already deleted, however in the verbosity of the 
previous pull-request this part was missed.  It may be worth noting that the 
standalone PdxAutoSerializer utility has its own antlr CMakeLists.txt.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/dgkimura/geode-native feature/GEODE-2441

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/geode-native/pull/33.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #33


commit 6357dc8c1784df06c789438e9b68fba493ce1a48
Author: David Kimura 
Date:   2017-02-24T19:54:59Z

GEODE-2441: Remove antlr reference from core sources




> 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)

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

2017-02-22 Thread ASF GitHub Bot (JIRA) 

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

ASF GitHub Bot commented on GEODE-2441:
---

Github user asfgit closed the pull request at:

https://github.com/apache/geode-native/pull/21


> 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)

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

2017-02-22 Thread ASF subversion and git services (JIRA) 

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

ASF subversion and git services commented on GEODE-2441:


Commit 97ded7036b08ce5d73b79c2786b01efdaa51d00f in geode-native's branch 
refs/heads/develop from [~dkimura]
[ https://git-wip-us.apache.org/repos/asf?p=geode-native.git;h=97ded70 ]

GEODE-2441: Move pdx auto serializer into standalone contrib

This closes #21.


> 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)

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

2017-02-21 Thread ASF GitHub Bot (JIRA) 

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

ASF GitHub Bot commented on GEODE-2441:
---

GitHub user dgkimura opened a pull request:

https://github.com/apache/geode-native/pull/21

GEODE-2441: Remove pdx auto serializer from core sources

Purpose of this pull-request is to simplify core dependencies and core 
source size by removing PdxAutoSerializer and related tests.  Reasoning is that 
PdxAutoSerializer is more of a standalone utility project that adds nearly 
40,000 lines of extra complexity.

A followup change will be to move PdxAutoSerializer as a stand-alone 
utility either into a `contrib` directory in `geode-native` repo or somewhere 
inside `geode-examples` repo.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/dgkimura/geode-native feature/GEODE-2441

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/geode-native/pull/21.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #21






> 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)

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

2017-02-21 Thread David Kimura (JIRA) 

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

David Kimura commented on GEODE-2441:
-

Current idea is to move PDXAutoSerializer either to geode-examples repo or into 
a utils directory (outside of normal build process) in geode-native.  Purpose 
of this JIRA is to reduce dependencies in the build process and extract 
non-core pieces of code from source tree.  In doing this, we hope that by 
reducing build complexity and core code size, the code will be easier to dive 
into and maintain.

> 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)

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

2017-02-10 Thread Avinash Dongre (JIRA) 

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

Avinash Dongre commented on GEODE-2441:
---

Just want to understand why we are removing PDXAutoSerializer utility ?

> 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)

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

2017-02-07 Thread Michael Dodge (JIRA) 

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

Michael Dodge commented on GEODE-2441:
--

Perhaps move the relevant code to a user-contributed portion of the repo?

> 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)