[geode-native] branch develop updated: GEODE-3453: Fix native client function tests to use updated API

This is an automated email from the ASF dual-hosted git repository. jbarrett pushed a commit to branch develop in repository https://gitbox.apache.org/repos/asf/geode-native.git The following commit(s) were added to refs/heads/develop by this push: new a2e1f10 GEODE-3453: Fix native client

Build failed in Jenkins: Geode-nightly #925

See Changes: [jdeppe] GEODE-3423: Have Gradle set LOCAL_USER_ID [ukohlmeyer] GEODE-3386 - Make KeyedErrorResponse & ErrorResponse siblings [ukohlmeyer] GEODE-3314 - Refactoring of DLockService to improve developer

[GitHub] geode pull request #716: GEODE-3406: Locator accepts Protobuf requests

Github user WireBaron commented on a diff in the pull request: https://github.com/apache/geode/pull/716#discussion_r133599245 --- Diff: geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ExecutionContext.java --- @@ -0,0 +1,54 @@ +/* + * Licensed to the

[GitHub] geode pull request #716: GEODE-3406: Locator accepts Protobuf requests

Github user WireBaron commented on a diff in the pull request: https://github.com/apache/geode/pull/716#discussion_r133598920 --- Diff: geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/operations/GetAvailableServersOperationHandler.java --- @@ -50,51 +37,19 @@

[GitHub] geode pull request #716: GEODE-3406: Locator accepts Protobuf requests

Github user WireBaron commented on a diff in the pull request: https://github.com/apache/geode/pull/716#discussion_r133596280 --- Diff: geode-core/src/main/java/org/apache/geode/distributed/internal/tcpserver/TcpServer.java --- @@ -334,42 +342,46 @@ protected void run() {

[GitHub] geode pull request #716: GEODE-3406: Locator accepts Protobuf requests

Github user WireBaron commented on a diff in the pull request: https://github.com/apache/geode/pull/716#discussion_r133597359 --- Diff: geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ClientProtoclMessageHandlerLoader.java --- @@ -0,0 +1,64 @@ +/* + *

[GitHub] geode pull request #716: GEODE-3406: Locator accepts Protobuf requests

Github user WireBaron commented on a diff in the pull request: https://github.com/apache/geode/pull/716#discussion_r133598026 --- Diff: geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ExecutionContext.java --- @@ -0,0 +1,54 @@ +/* + * Licensed to the

[GitHub] geode pull request #716: GEODE-3406: Locator accepts Protobuf requests

Github user WireBaron commented on a diff in the pull request: https://github.com/apache/geode/pull/716#discussion_r133595959 --- Diff: geode-core/src/main/java/org/apache/geode/distributed/internal/tcpserver/TcpServer.java --- @@ -120,6 +126,7 @@ private InetAddress bind_a

[GitHub] geode pull request #716: GEODE-3406: Locator accepts Protobuf requests

Github user WireBaron commented on a diff in the pull request: https://github.com/apache/geode/pull/716#discussion_r133599146 --- Diff: geode-protobuf/src/test/java/org/apache/geode/protocol/protobuf/operations/GetAvailableServersOperationHandlerJUnitTest.java --- @@ -32,75 +30,48

[GitHub] geode pull request #716: GEODE-3406: Locator accepts Protobuf requests

Github user WireBaron commented on a diff in the pull request: https://github.com/apache/geode/pull/716#discussion_r133596604 --- Diff: geode-core/src/main/java/org/apache/geode/internal/cache/InternalCache.java --- @@ -76,7 +76,9 @@ */ public interface InternalCache ext

[GitHub] geode pull request #716: GEODE-3406: Locator accepts Protobuf requests

Github user WireBaron commented on a diff in the pull request: https://github.com/apache/geode/pull/716#discussion_r133597935 --- Diff: geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ExecutionContext.java --- @@ -0,0 +1,54 @@ +/* + * Licensed to the

Re: Review Request 61672: GEODE-3249: internal messages should require credentials

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61672/#review183089 --- Fix it, then Ship it! geode-core/src/main/java/org/apache/geod

[GitHub] geode pull request #716: GEODE-3406: Locator accepts Protobuf requests

GitHub user pivotal-amurmann opened a pull request: https://github.com/apache/geode/pull/716 GEODE-3406: Locator accepts Protobuf requests Also addresses GEODE-3400, GEODE-3399 This allows the locator to respond to Protobuf requests. Currently it will only be able to respond

Re: Review Request 61676: Release the lock held when beforeCompletion failed with CommitConflictException

> On Aug. 16, 2017, 10:47 p.m., anilkumar gingade wrote: > > geode-core/src/main/java/org/apache/geode/internal/cache/TXState.java > > Line 1024 (original), 1024 (patched) > > > > > > And remove this one too... Yes

Re: How to export region keys in json file

There isn't anything built in to export keys as JSON. There was some talk about exporting values as JSON at one time, but that hasn't been done. I attached a function to dump the keys as JSON. It just logs them to the terminal, but it can easily be changed to write them to a file. It dumps output

Re: Review Request 61676: Release the lock held when beforeCompletion failed with CommitConflictException

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61676/#review183078 --- geode-core/src/main/java/org/apache/geode/internal/cache/TXState.

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

--- Spring Data GemFire > Nightly-ApacheGeode > #649 was successful. --- Scheduled 2029 tests in total. https://build.spring.io/browse/SGF-NAG-649/ -- This

[GitHub] geode pull request #715: GEODE-3169: Decoupling of DiskStore and backups

GitHub user nreich opened a pull request: https://github.com/apache/geode/pull/715 GEODE-3169: Decoupling of DiskStore and backups * move backup logic away from DiskStore and into BackupManager * refactor code into smaller methods * improve test code clarity You can me

Review Request 61701: GEODE-3277: Fix error path constructors of Launcher inner State classess

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61701/ --- Review request for geode, Emily Yeh, Jinmei Liao, Jared Stewart, Kirk Lund, and

Re: Review Request 61627: GEODE-3437: Fix list and describe region tests

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61627/ --- (Updated Aug. 16, 2017, 8:44 p.m.) Review request for geode, Emily Yeh, Jared S

Re: Review Request 61694: GEODE-3235: Deploy jar registers functions which extend FunctionAdapter

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61694/#review183049 --- geode-core/src/main/java/org/apache/geode/internal/DeployedJar.ja

Re: Review Request 61694: GEODE-3235: Deploy jar registers functions which extend FunctionAdapter

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61694/#review183048 --- It looks like FunctionAdapter has bee deprecated for around a year

Re: Review Request 61627: GEODE-3437: Fix list and describe region tests

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61627/#review183046 --- Ship it! Ship It! - Patrick Rhomberg On Aug. 16, 2017, 5:24

Review Request 61694: GEODE-3235: Deploy jar registers functions which extend FunctionAdapter

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

RE: continuous query internal mechanism questions

It seems like the code in the native client (in the version I have, which may be old) send the message to all servers: CqResultsPtr CqQueryImpl::executeWithInitialResults(uint32_t timeout) { ... TcrMessage msg(TcrMessage::EXECUTECQ_WITH_IR_MSG_TYPE, m_cqName, m_queryString, CqState::RUN

Re: Review Request 61627: GEODE-3437: Fix list and describe region tests

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61627/#review183042 --- Ship it! Ship It! - Ken Howe On Aug. 16, 2017, 5:24 p.m., Ja

Re: Review Request 61627: GEODE-3437: Fix list and describe region tests

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61627/ --- (Updated Aug. 16, 2017, 5:24 p.m.) Review request for geode, Emily Yeh, Jared S

[GitHub] geode pull request #702: GEODE-3416: Reduce synchronization blockages in Soc...

Github user hiteshk25 commented on a diff in the pull request: https://github.com/apache/geode/pull/702#discussion_r133506959 --- Diff: geode-core/src/main/java/org/apache/geode/internal/net/SocketCloser.java --- @@ -96,46 +99,55 @@ public int getMaxThreads() { return this

RE: continuous query internal mechanism questions

Thanks, The subscription-redundancy is set to "1" and the region is used in 2 nodes (there are more nodes which re not related to it). Yes there is an exception, which I am yet to understand: (and this exception causes the closure of the CQ in this node as well as sending operation message to t

Re: Review Request 61627: GEODE-3437: Fix list and describe region tests

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61627/#review183037 --- Ship it! Ship It! - Ken Howe On Aug. 14, 2017, 10:40 p.m., J

Re: Build failed in Jenkins: Geode-nightly #923

I'm working on enabling parallel dunits in docker to work on Jenkins. If this can happen it should help with some of these problems. However, it requires fairly beefy systems to run well, otherwise you end up with other issues around resource starvation. At worst we could just run with a parallelis

Re: ACTION REQUIRED: geode-native has moved

Nope! Sent from my iPhone > On Aug 16, 2017, at 7:33 AM, Michael William Dodge wrote: > > Does this affect a pre-existing fork? > > Sarge > >> On 16 Aug, 2017, at 07:24, Jacob Barrett wrote: >> >> Devs, >> >> The geode-native repo has moved to GitHub utilizing the Apache GitBox >> service.

Re: ACTION REQUIRED: geode-native has moved

Does this affect a pre-existing fork? Sarge > On 16 Aug, 2017, at 07:24, Jacob Barrett wrote: > > Devs, > > The geode-native repo has moved to GitHub utilizing the Apache GitBox > service. > > For contributors nothing really changes, the repo is still at > github.com/apache/geode-native. > >

ACTION REQUIRED: geode-native has moved

Devs, The geode-native repo has moved to GitHub utilizing the Apache GitBox service. For contributors nothing really changes, the repo is still at github.com/apache/geode-native. For committers the origin is now github.com/apache/geode-native. Before you can get write access to the new repo you

How to export region keys in json file

Hi, How I can export Region keys in json file. It will help full for us to use get command or remove command for particular key when we have the composite keys defined. get --region=/CUSTOMER_1 --key=({'customerID':'1593','customerGroup':'30013','cycleCode':'1','partitionID':'300133'}) --key-