RE: [GitHub] incubator-geode pull request #199: GEODE-88: code fixes for c++ client

2016-07-26 Thread Dor Ben Dov
Comment will be remove. Dor -Original Message- From: Anthony Baker [mailto:aba...@pivotal.io] Sent: יום ג 26 יולי 2016 18:22 To: dev@geode.incubator.apache.org Subject: Re: [GitHub] incubator-geode pull request #199: GEODE-88: code fixes for c++ client Based on my reading of [1] and [2

Re: [GitHub] incubator-geode pull request #199: GEODE-88: code fixes for c++ client

2016-07-26 Thread Anthony Baker
Based on my reading of [1] and [2] I don’t think the comment is necessary and should be removed. Any other opinions? Anthony [1] http://www.apache.org/legal/src-headers.html [2] http://www.apache.org/dev/apply-license.html#contributor-copyright > On Jul 26, 2016, at 12:38 AM, doribd

RE: [GitHub] incubator-geode pull request #199: GEODE-88: code fixes for c++ client

2016-07-19 Thread Gal Palmery
Yes, I can add the call to ACE::init() if some specific environment variable is not defined. -Original Message- From: metatype [mailto:g...@git.apache.org] Sent: Monday, July 18, 2016 18:32 To: dev@geode.incubator.apache.org Subject: [GitHub] incubator-geode pull request #199: