Re: Review Request 60106: GEODE-3052 Restarting 2 locators within 1s of each other causes potential locator split brain

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60106/#review178617 --- Ship it! Ship It! - Hitesh Khamesra On June 21, 2017, 10:24

Re: Review Request 60106: GEODE-3052 Restarting 2 locators within 1s of each other causes potential locator split brain

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60106/#review178595 --- not sure you saw following commant, as I was having trouble with

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

Github user galen-pivotal commented on the issue: https://github.com/apache/geode/pull/586 This failed due to Spotless; closing in favor of #597. --- 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 project does not

[GitHub] geode pull request #586: GEODE-3075: initial flow adding the new client prot...

Github user galen-pivotal closed the pull request at: https://github.com/apache/geode/pull/586 --- 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 project does not have this feature enabled and wishes so, or if the feature

Re: Review Request 60347: GEODE-3097: GFSH works over HTTP with SSL

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60347/#review178604 --- Ship it! Ship It! - Jinmei Liao On June 21, 2017, 10:48

Re: Review Request 60106: GEODE-3052 Restarting 2 locators within 1s of each other causes potential locator split brain

> On June 21, 2017, 11:05 p.m., Hitesh Khamesra wrote: > > geode-core/src/main/java/org/apache/geode/distributed/internal/membership/gms/membership/GMSJoinLeave.java > > Lines 710 (patched) > > > > > > In which

[GitHub] geode pull request #591: GEODE-3073: Renamed OrderByComparatorUnmapped to Or...

Github user jhuynh1 commented on a diff in the pull request: https://github.com/apache/geode/pull/591#discussion_r123388223 --- Diff: geode-core/src/main/java/org/apache/geode/cache/query/internal/CompiledSelect.java --- @@ -925,79 +917,89 @@ private SelectResults

Re: Review Request 60106: GEODE-3052 Restarting 2 locators within 1s of each other causes potential locator split brain

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60106/#review178585 ---

Re: Review Request 60106: GEODE-3052 Restarting 2 locators within 1s of each other causes potential locator split brain

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60106/#review178585 ---

Review Request 60348: GEODE-3103: GfshRule no longer clutters output

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

Review Request 60347: GEODE-3097: GFSH works over HTTP with SSL

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

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

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

Re: Review Request 60278: GEODE-3101 fix local locks not being released in client if the JTA transaction failed due to commit conflict on server in beforeCompletion()

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60278/#review178577 --- Ship it! Ship It! - anilkumar gingade On June 21, 2017,

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

Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/588 --- 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 project does not have this feature enabled and wishes so, or if the feature is

Re: Review Request 60106: GEODE-3052 Restarting 2 locators within 1s of each other causes potential locator split brain

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60106/#review178573 ---

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

We've been seeing this since mid-May from Jenkins ... is someone following up on this, or should I open a new ticket for this? [error 2017/06/20 19:06:27.924 UTC tid=0x13] failed setting interface to /127.0.1.1: java.net.SocketException: bad argument for IP_MULTICAST_IF: address not bound to

Review Request 60312: GEODE-2804 Cache InetAddress if configure host as ip string.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60312/ --- Review request for geode, Alexander Murmann, Bruce Schuchardt, and Galen

[GitHub] geode issue #596: GEODE-2920 - GEODE-2925: Finer Grained Security

Github user PurelyApplied commented on the issue: https://github.com/apache/geode/pull/596 Precheckin running. --- 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 project does not have this feature enabled and wishes

[GitHub] geode pull request #596: GEODE-2920 - GEODE-2925: Finer Grained Security

GitHub user PurelyApplied opened a pull request: https://github.com/apache/geode/pull/596 GEODE-2920 - GEODE-2925: Finer Grained Security Due to the size of this commit and for your convenience of review, I have not yet squashed my commits. Do note that I have not individually

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

Github user galen-pivotal commented on the issue: https://github.com/apache/geode/pull/586 @bschuchardt Was that the only failure? It looks like another test had set `mcast-port` to a non-zero value and failed to clean up the environment I'm rerunning precheckin; will

[GitHub] geode pull request #576: Geode 2920, 2921, 2922, 2924

Github user PurelyApplied closed the pull request at: https://github.com/apache/geode/pull/576 --- 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 project does not have this feature enabled and wishes so, or if the feature

[GitHub] geode issue #576: Geode 2920, 2921, 2922, 2924

Github user PurelyApplied commented on the issue: https://github.com/apache/geode/pull/576 Closing this PR to reopen the finalized form. --- 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 project does not have this

Re: Need information about FunctionStatistics

We had plans to create a new Statistics API [1] for people to use to subscribe to stats and expose them on custom user-implemented MBeans. The use case matches what you want to do. Unfortunately without the completion of that new API, what you're trying to do isn't going to be easy and would

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60217/#review178541 --- Ship it! Ship It! - Galen O'Sullivan On June 21, 2017, 7:01

Re: Need information about FunctionStatistics

This will be a nice addition. I'm not sure I would create the MBeans for all the registered functions at once. You'll have to deal with functions being registered on the fly or after you've called handleFunctionCreation. I think I would change the handleFunctionCreation to take a Function or

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60217/#review178527 --- Ship it! Ship It! - Hitesh Khamesra On June 21, 2017, 7:01

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60217/ --- (Updated June 21, 2017, 7:01 p.m.) Review request for geode, Alexander

[GitHub] geode pull request #589: GEODE-393: Providing cache for FunctionContext

Github user jinmeiliao commented on a diff in the pull request: https://github.com/apache/geode/pull/589#discussion_r123332996 --- Diff: geode-core/src/main/java/org/apache/geode/internal/cache/execute/FunctionContextImpl.java --- @@ -37,20 +38,25 @@ private String

[GitHub] geode pull request #578: GEODE-1958: Removing/deprecating PasswordUtil

Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/578 --- 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 project does not have this feature enabled and wishes so, or if the feature is

Re: Query for JMX "Async Event Queue properties"

I don't see that OR clause on Geode develop. What version of Geode are you using? In any event, the JMX MBeans are registered using a ResourceEventsListener called ManagementListener. That listener is invoked by handleResourceEvent calls in GemFireCacheImpl for a bunch of different events

Review Request 60278: GEODE-3101 fix local locks not being released in client if the JTA transaction failed due to commit conflict on server in beforeCompletion()

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60278/ --- Review request for geode, anilkumar gingade, Lynn Gallinat, and Swapnil

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60217/#review178513 ---

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

> On June 21, 2017, 12:52 a.m., Udo Kohlmeyer wrote: > > geode-protobuf/src/main/java/org/apache/geode/protocol/operations/registry/OperationsHandlerRegistry.java > > Lines 24 (patched) > > > > > > It is hard to say

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

Github user bschuchardt commented on the issue: https://github.com/apache/geode/pull/586 There was a failure in one of the new tests :geode-core:integrationTest org.apache.geode.internal.cache.tier.sockets.ServerConnectionFactoryIntegrationTest >

Jenkins build is back to normal : Geode-nightly #873

See

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

Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/594 --- 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 project does not have this feature enabled and wishes so, or if the feature is

[GitHub] geode pull request #595: GEODE-3104 : Modified code to use --include-locator...

Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/595 --- 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 project does not have this feature enabled and wishes so, or if the feature is

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60257/ --- (Updated June 21, 2017, 3 p.m.) Review request for geode, Emily Yeh, Jared

RE: Need information about FunctionStatistics

Hi , I have registered all function stats to JMX by creating new bean. But need to know what Is the correct place register this bean. I did like this : public void handleFunctionCreation() throws ManagementException { Map functionMap =

OPEN YOUR ACCESS SYSTEM

Open your access Data register from indonesian Model : smj-110G ( samsung J1 ace ) Imei : 354028079052893 Os : Android 4.4.4 Pact level : 2016-02-01 Baseband : J110GDXU0APB1 Build private: KTU84PJ110GDXU0APB1 K - version : 3.14.0 dpi@SWHD7323 #1 Mon feb 22 19:13:26 KST 2016 Please open your

OPEN YOUR ACCESS SYSTEM

Open your access Data register from indonesian Model : smj-110G ( samsung J1 ace ) Imei : 354028079052893 Os : Android 4.4.4 Pact level : 2016-02-01 Baseband : J110GDXU0APB1 Build private: KTU84PJ110GDXU0APB1 K - version : 3.14.0 dpi@SWHD7323 #1 Mon feb 22 19:13:26 KST 2016 Please open your

OPEN YOUR ACCESS SYSTEM

Open your access Data register from indonesian Model : smj-110G ( samsung J1 ace ) Imei : 354028079052893 Os : Android 4.4.4 Pact level : 2016-02-01 Baseband : J110GDXU0APB1 Build private: KTU84PJ110GDXU0APB1 K - version : 3.14.0 dpi@SWHD7323 #1 Mon feb 22 19:13:26 KST 2016 Please open your

[GitHub] geode pull request #595: GEODE-3104 : Modified code to use --include-locator...

GitHub user ameybarve15 opened a pull request: https://github.com/apache/geode/pull/595 GEODE-3104 : Modified code to use --include-locators instead of existing --shutdown-locators option in shutdown command. GEODE-3104 : Modified code to use --include-locators instead of existing