Build failed in Jenkins: Geode-nightly-flaky #37

2017-06-16 Thread Apache Jenkins Server
See Changes: [eshu] GEODE-2301: Deprecate JTA transaction manager from Geode [boglesby] GEODE-3072: Changed getMembershipId to use the client version [jiliao] GEODE-2632: consolidate different types of

Build failed in Jenkins: Geode-nightly #868

2017-06-16 Thread Apache Jenkins Server
See Changes: [eshu] GEODE-2301: Deprecate JTA transaction manager from Geode [boglesby] GEODE-3072: Changed getMembershipId to use the client version [jiliao] GEODE-2632: consolidate different types of

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

2017-06-16 Thread YehEmily
GitHub user YehEmily opened a pull request: https://github.com/apache/geode/pull/591 GEODE-3073: Renamed OrderByComparatorUnmapped to OrderByComparatorMapped [View the JIRA ticket here.](https://issues.apache.org/jira/browse/GEODE-3073) `OrderByComparatorUnmapped.java`

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

2017-06-16 Thread DivineEnder
Github user DivineEnder commented on the issue: https://github.com/apache/geode/pull/589 @ladyVader @nabarunnag @boglesby @jhuynh1 @upthewaterspout @gesterzhou --- 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

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

2017-06-16 Thread DivineEnder
GitHub user DivineEnder opened a pull request: https://github.com/apache/geode/pull/589 GEODE-393: Providing cache for FunctionContext Added the ability to pass a Cache to a function context as well as tests to check for this. Thank you for submitting a contribution to

[GitHub] geode pull request #590: GEODE-3090: Fixing gfsh help message (and a lot of ...

2017-06-16 Thread YehEmily
GitHub user YehEmily opened a pull request: https://github.com/apache/geode/pull/590 GEODE-3090: Fixing gfsh help message (and a lot of other typos) [View the original JIRA ticket here.](https://issues.apache.org/jira/browse/GEODE-3090) The original ticket called for

Broken: apache/geode#2906 (develop - 4bf80a6)

2017-06-16 Thread Travis CI
Build Update for apache/geode - Build: #2906 Status: Broken Duration: 9 minutes and 59 seconds Commit: 4bf80a6 (develop) Author: Barry Oglesby Message: GEODE-3072: Ignore dunit test View the changeset:

[GitHub] geode pull request #583: GEODE-2854: GEODE REST API Docs Missing Authenticat...

2017-06-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/583 --- 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

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

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

Re: Error running category tests due to RawLuceneRepositoryManagerJUnitTest

2017-06-16 Thread Kirk Lund
Or something like this for "Project compiler output" so that IntelliJ isn't defaulting to overwriting the same class files as command-line gradle is: File -> Project Structure -> Project -> Project compiler output: /Users/klund/dev/geode/build-intellij/classes Having IntelliJ output classes to

Re: Error running category tests due to RawLuceneRepositoryManagerJUnitTest

2017-06-16 Thread Kirk Lund
I haven't seen this exact problem, but I've seen weird behavior if the command-line gradle build terminated before completing. I suggest completing a clean build on the command-line, then restart IntelliJ and maybe also edit Project Structure to tell IntelliJ to output classes to a different

Error running category tests due to RawLuceneRepositoryManagerJUnitTest

2017-06-16 Thread Bruce Schuchardt
Has anyone else seen this in IntelliJ? Exception in thread "main" java.lang.VerifyError: Bad type on operand stack Exception Details: Location: org/apache/geode/cache/lucene/internal/RawLuceneRepositoryManagerJUnitTest.createIndexAndRepoManager()V @7: putstatic Reason: Type

Re: Query for JMX "Async Event Queue properties"

2017-06-16 Thread Barry Oglesby
The AsynceEventQueueMBean is monitoring the AsyncEventQueueStats not the PartitionedRegionStats. And, actually, the only stat being monitored by that MBean is the eventQueueSize stat. The parallel AsyncEventQueue creates a colocated PartitionedRegion under the covers, and the stats for that PR are

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

2017-06-16 Thread nabarunnag
GitHub user nabarunnag opened a pull request: https://github.com/apache/geode/pull/588 GEODE-2820: Added awaitlity clause to wait for the region size to be … …correct * Added awaitility clause to wait for all the regions to have all the entries * This is

[GitHub] geode issue #587: GEODE-2018: Added instantiators to SessionCachingFilter in...

2017-06-16 Thread DivineEnder
Github user DivineEnder commented on the issue: https://github.com/apache/geode/pull/587 @ladyVader @nabarunnag @boglesby @jhuynh1 @upthewaterspout @gesterzhou --- 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

[GitHub] geode pull request #587: GEODE-2018: Added instantiators to SessionCachingFi...

2017-06-16 Thread DivineEnder
GitHub user DivineEnder opened a pull request: https://github.com/apache/geode/pull/587 GEODE-2018: Added instantiators to SessionCachingFilter init method Also removed unused registerInstantiator methods from GatewayDeltaCreateEvent, GatewayDeltaDestroyEvent, and

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

2017-06-16 Thread Galen O'Sullivan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60157/ --- Review request for geode, Alexander Murmann, Bruce Schuchardt, Hitesh Khamesra,

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

2017-06-16 Thread galen-pivotal
GitHub user galen-pivotal opened a pull request: https://github.com/apache/geode/pull/586 GEODE-3075: initial flow adding the new client protocol to core. Create `ServerConnectionFactory`, which creates either instances of LegacyServerConnection (identical in functionality to the

[GitHub] geode pull request #585: GEODE-3091: remove empty method

2017-06-16 Thread nreich
GitHub user nreich opened a pull request: https://github.com/apache/geode/pull/585 GEODE-3091: remove empty method You can merge this pull request into a Git repository by running: $ git pull https://github.com/nreich/geode feature/GEODE-3091 Alternatively you can review and

[GitHub] geode pull request #581: GEODE-2301 Doc note to deprecate Geode JTA trans mg...

2017-06-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/581 --- 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 60125: GEODE-3062: replace SecurityService after loading cluster config [FIXED]

2017-06-16 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60125/#review178114 --- Ship it! Ship It! - Jinmei Liao On June 16, 2017, 4:23

[GitHub] geode issue #581: GEODE-2301 Doc note to deprecate Geode JTA trans mgr

2017-06-16 Thread joeymcallister
Github user joeymcallister commented on the issue: https://github.com/apache/geode/pull/581 +1 LGTM --- 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

[GitHub] geode issue #584: GEODE-2901: Adding integration tests of session replicatio...

2017-06-16 Thread upthewaterspout
Github user upthewaterspout commented on the issue: https://github.com/apache/geode/pull/584 +1 - Looks good to me! --- 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

Re: Review Request 60125: GEODE-3062: replace SecurityService after loading cluster config [FIXED]

2017-06-16 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60125/ --- (Updated June 16, 2017, 4:23 p.m.) Review request for geode, Emily Yeh, Jinmei

[GitHub] geode pull request #582: GEODE-2601: Fix banner being logged twice

2017-06-16 Thread YehEmily
Github user YehEmily commented on a diff in the pull request: https://github.com/apache/geode/pull/582#discussion_r122476764 --- Diff: geode-core/src/main/java/org/apache/geode/distributed/internal/InternalDistributedSystem.java --- @@ -2167,7 +2191,7 @@ private static void

Re: Review Request 60125: GEODE-3062: replace SecurityService after loading cluster config [FIXED]

2017-06-16 Thread Kirk Lund
> On June 15, 2017, 11:47 p.m., Jinmei Liao wrote: > > geode-core/src/test/java/org/apache/geode/test/dunit/rules/ServerStarterRule.java > > Lines 132 (patched) > > > > > > I am surprised that server.stop() does not

Re: Review Request 60142: GEODE-3071: Provide capability to parallelize distributedTests

2017-06-16 Thread Jens Deppe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60142/ --- (Updated June 16, 2017, 3:07 p.m.) Review request for geode, Anthony Baker,

Query for JMX "Async Event Queue properties"

2017-06-16 Thread Dinesh Akhand
Hi Team Regarding the following metrics: /AsyncEventQueue_XYZ_PARALLEL_GATEWAY_SENDER_QUEUE:PartitionedRegionStats.dataStoreEntryCount /AsyncEventQueue_XYZ_PARALLEL_GATEWAY_SENDER_QUEUE:PartitionedRegionStats.dataStoreBytesInUse I found only count for async queue in JMX , it called here

Re: Review Request 60142: GEODE-3071: Provide capability to parallelize distributedTests

2017-06-16 Thread Mark Bretl
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60142/#review178086 --- Ship it! Excellent work Jens! Only issue I see is to add