Re: Individual Column update using function

2017-06-20 Thread Akihiro Kitada
Hello Vigneshwaran, >I am struggling to update individual column using function. In this context, does "individual column" mean the specific field variable of the domain object stored in the target region, below? - public class EmployeeData { private static final long serialVersionUID = 1L;

Re: Review Request 60217: GEODE-2995: adding in protocol handler for protobuf messages

2017-06-20 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60217/#review178442 --- geode-protobuf/src/main/java/org/apache/geode/ProtobufUtilities.j

Re: Review Request 60257: GEODE-3100/GEM-1256: move authorize* call to external SecurityService and have it available in Cache

2017-06-20 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60257/ --- (Updated June 21, 2017, 12:47 a.m.) Review request for geode, Emily Yeh, Jared

Re: Review Request 60257: GEODE-3100/GEM-1256: move authorize* call to external SecurityService and have it available in Cache

2017-06-20 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60257/ --- (Updated June 21, 2017, 12:45 a.m.) Review request for geode, Emily Yeh, Jared

Review Request 60257: GEODE-3100/GEM-1256: move authorize* call to external SecurityService and have it available in Cache

2017-06-20 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60257/ --- Review request for geode, Emily Yeh, Jared Stewart, Ken Howe, Kirk Lund, and Pat

[GitHub] geode issue #588: GEODE-2820: Added awaitlity clause to wait for the region ...

2017-06-20 Thread nabarunnag
Github user nabarunnag commented on the issue: https://github.com/apache/geode/pull/588 @jhuynh1 Changed the validation checks to verify that the size of the indexes are correct. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub a

Re: Review Request 60217: GEODE-2995: adding in protocol handler for protobuf messages

2017-06-20 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60217/#review178441 --- geode-protobuf/build.gradle Lines 26 (patched)

Jenkins build is back to normal : Geode-nightly-flaky #41

2017-06-20 Thread Apache Jenkins Server
See

Re: Review Request 60217: GEODE-2995: adding in protocol handler for protobuf messages

2017-06-20 Thread Brian Rowe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60217/ --- (Updated June 21, 2017, 12:02 a.m.) Review request for geode, Alexander Murmann

Re: Review Request 60217: GEODE-2995: adding in protocol handler for protobuf messages

2017-06-20 Thread Brian Rowe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60217/#review178412 --- geode-protobuf/src/main/java/org/apache/geode/protocol/OpsProcess

Re: Review Request 60157: GEODE-3075: initial work for feature flag and creation of a new subclass of `ServerConnection`.

2017-06-20 Thread Udo Kohlmeyer
> On June 19, 2017, 4:24 p.m., Udo Kohlmeyer wrote: > > geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/NewProtocolServerConnection.java > > Lines 54 (patched) > > > > > > could we rename this to

Build failed in Jenkins: Geode-release #69

2017-06-20 Thread Apache Jenkins Server
See Changes: [jiliao] GEODE-3092: fix specifiedDefaultValue for cacheLoader and cacheWriter [jiliao] GEODE-3095: fix parameter type mismatch between the diskstore command [jiliao] GEODE-3092: fix specifiedDefaultVal

[Spring CI] Spring Data GemFire > Nightly-ApacheGeode > #591 was SUCCESSFUL (with 1868 tests)

2017-06-20 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #591 was successful. --- Scheduled 1870 tests in total. https://build.spring.io/browse/SGF-NAG-591/ -- This

Build failed in Jenkins: Geode-nightly #872

2017-06-20 Thread Apache Jenkins Server
See Changes: [jdeppe] GEODE-3071: Provide capability to parallelize distributedTests [jdeppe] Remove debug println [jdeppe] GEODE_3071: Add Apache license header [jdeppe] GEODE-3071: Pull config of docker volumes

Re: Review Request 60217: GEODE-2995: adding in protocol handler for protobuf messages

2017-06-20 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60217/#review178410 --- geode-protobuf/src/main/java/org/apache/geode/ProtobufUtilities.j

[GitHub] geode issue #586: GEODE-3075: initial flow adding the new client protocol to...

2017-06-20 Thread bschuchardt
Github user bschuchardt commented on the issue: https://github.com/apache/geode/pull/586 I'm running precheckin on this PR & will merge it to develop when finished --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your p

Individual Column update using function

2017-06-20 Thread vigneshwaran02
I am struggling to update individual column using function. Can anyone share code snippet to handle at function side to update. Region - Partition Client code - Map val = new HashMap(); val.put("lastName", "G"); Set setFilter = new HashSet();

Re: Review Request 60217: GEODE-2995: adding in protocol handler for protobuf messages

2017-06-20 Thread Galen O'Sullivan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60217/#review178419 --- geode-protobuf/src/test/java/org/apache/geode/protocol/handler/Pr

Re: [VOTE] Using Pull Requests over Review Board

2017-06-20 Thread Ernest Burghardt
+1 for PRs in Github - one place, simple On Wed, Jun 14, 2017 at 9:17 AM, Kenneth Howe wrote: > -1 per my comments on the [DISCUSS] thread > > > On Jun 11, 2017, at 9:51 AM, Jacob Barrett wrote: > > > > After a few days of discussion [1] this thread has quieted down so I > would > > like to tak

[GitHub] geode pull request #588: GEODE-2820: Added awaitlity clause to wait for the ...

2017-06-20 Thread jhuynh1
Github user jhuynh1 commented on a diff in the pull request: https://github.com/apache/geode/pull/588#discussion_r123060011 --- Diff: geode-core/src/test/java/org/apache/geode/cache/query/dunit/QueryIndexUsingXMLDUnitTest.java --- @@ -510,6 +512,16 @@ public void testCreateAsyncI

Re: Review Request 60217: GEODE-2995: adding in protocol handler for protobuf messages

2017-06-20 Thread Galen O'Sullivan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60217/#review178404 --- Good work! I would love to see more doc comments in general. It c

Re: Review Request 60157: GEODE-3075: initial work for feature flag and creation of a new subclass of `ServerConnection`.

2017-06-20 Thread Galen O'Sullivan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60157/ --- (Updated June 20, 2017, 5:21 p.m.) Review request for geode, Alexander Murmann,

Re: release 1.2

2017-06-20 Thread Dave Barnes
I have staged a v1.2 doc update for the Apache Geode website, using RC 1 versions of the User Guide and the Javadocs. Later release candidates are unlikely to require doc changes, other than the Wiki-based relnotes, so I'm nearly certain this will be the final version. When we merge or rename this

[CANCEL][VOTE] Apache Geode release - v1.2.0 RC1

2017-06-20 Thread Anthony Baker
This vote is cancelled due to GEODE-3095, GEODE-3092. Anthony > On Jun 17, 2017, at 2:57 PM, Anthony Baker wrote: > > Thanks Mike! Here’s the corrected text: > > This is the first release candidate for Apache Geode, version > 1.2.0. Thanks to all the community members for their > contributio

Re: Review Request 60157: GEODE-3075: initial work for feature flag and creation of a new subclass of `ServerConnection`.

2017-06-20 Thread Alexander Murmann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60157/#review178396 --- Ship it! Ship It! - Alexander Murmann On June 20, 2017, 12:5

Re: Review Request 60157: GEODE-3075: initial work for feature flag and creation of a new subclass of `ServerConnection`.

2017-06-20 Thread Alexander Murmann
> On June 20, 2017, 3:54 p.m., Alexander Murmann wrote: > > All my issues are minor. Overall this looks good to me. Maybe we can consider some of my comments in future refactors. The biggest issue I see is the security logging. - Alexander --

Re: Review Request 60157: GEODE-3075: initial work for feature flag and creation of a new subclass of `ServerConnection`.

2017-06-20 Thread Alexander Murmann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60157/#review178387 --- geode-core/src/main/java/org/apache/geode/internal/cache/tier/soc

[GitHub] geode pull request #594: GEODE-2861: Remove dead code

2017-06-20 Thread nreich
GitHub user nreich opened a pull request: https://github.com/apache/geode/pull/594 GEODE-2861: Remove dead code * code removed related to shutting down DiskStoreTaskPool Thank you for submitting a contribution to Apache Geode. In order to streamline the review of

RE: Query for JMX "Async Event Queue properties"

2017-06-20 Thread Dinesh Akhand
Hi Barry, Looks like things are already implemented in geode: Class : GemFireCacheImpl if (!rgn.isInternalRegion() || rgn.getFullPath().contains("GATEWAY_SENDER_QUEUE")) { system.handleResourceEvent(ResourceEvent.REGION_CREATE, rgn); } Above code solved my problem . Async