[GitHub] geode pull request #619: GEODE-2901: Refining session replication tests

2017-07-10 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/619 --- 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 enabl

Re: slf4j-jdk

2017-07-10 Thread David Anuta
It looks like that might have been caused by my commit for GEODE-2901. slf4j-jdk.jar is included in the Tomcat zip folder located within the geode-assembly/build/install/apache-geode/tools/Modules/ modules directory and the change extracts that folder. I've created GEODE-3179

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

2017-07-10 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #612 was successful. --- Scheduled 1910 tests in total. https://build.spring.io/browse/SGF-NAG-612/ -- This

[GitHub] geode issue #622: GEODE-1731: Modifying region size check to be more accurat...

2017-07-10 Thread ladyVader
Github user ladyVader commented on the issue: https://github.com/apache/geode/pull/622 Looks good! --- 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 th

[GitHub] geode issue #622: GEODE-1731: Modifying region size check to be more accurat...

2017-07-10 Thread DivineEnder
Github user DivineEnder commented on the issue: https://github.com/apache/geode/pull/622 @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 proj

[GitHub] geode pull request #623: GEODE-3156: add AcceptanceTest gradle target and ju...

2017-07-10 Thread kirklund
GitHub user kirklund opened a pull request: https://github.com/apache/geode/pull/623 GEODE-3156: add AcceptanceTest gradle target and junit category Thank you for submitting a contribution to Apache Geode. In order to streamline the review of the contribution we ask you

[GitHub] geode pull request #622: GEODE-1731: Modifying region size check to be more ...

2017-07-10 Thread DivineEnder
GitHub user DivineEnder opened a pull request: https://github.com/apache/geode/pull/622 GEODE-1731: Modifying region size check to be more accurate Changed the hardcoded 5010 region size check to the size of the region already plus 20 new elements. The 20 new elements is due to the

Re: Review Request 60666: geode-3166: remove the uncalled getCredential method

2017-07-10 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60666/#review180088 --- geode-core/src/main/java/org/apache/geode/security/AuthInitialize

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

2017-07-10 Thread Kirk Lund
> On June 29, 2017, 5:34 p.m., Kirk Lund wrote: > > geode-core/src/test/java/org/apache/geode/management/ConnectToLocatorSSLDUnitTest.java > > Line 63 (original), 66 (patched) > > > > > > Looks like this test has some

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

2017-07-10 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60347/#review180091 --- geode-core/src/test/java/org/apache/geode/management/ConnectToLoc

Re: Review Request 60202: GEODE-3056: fix the message for invalid partition-resolver

2017-07-10 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60202/#review180089 --- Ship it! Ship It! - Kirk Lund On June 19, 2017, 7:08 p.m., J

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

2017-07-10 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60257/#review180087 --- geode-core/src/main/java/org/apache/geode/cache/Cache.java Lines

Re: getSizeInBytes() return type

2017-07-10 Thread Michael Stolz
I don't think it is appropriate to store objects of 2GB size in Geode. I know we have hit networking issues with such large objects in the past. -- Mike Stolz Principal Engineer, GemFire Product Manager Mobile: +1-631-835-4771 On Sun, Jul 9, 2017 at 2:13 AM, Daniel Farcovich < daniel.farcov...@am

Failed: jinmeiliao/geode#11 (query2 - adb393d)

2017-07-10 Thread Travis CI
Build Update for jinmeiliao/geode - Build: #11 Status: Failed Duration: 27 minutes and 42 seconds Commit: adb393d (query2) Author: Jinmei Liao Message: create a query2 command View the changeset: https://github.com/jinmeiliao/geode/compare/e715abfb9fc1^...adb

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

2017-07-10 Thread Apache Jenkins Server
See Changes: [eshu] GEODE-3015: Fix a flaky test by adding wait. -- [...truncated 98.18 KB...] Download https://repo1.maven.org/maven2/io/springfox/springfox-spring-web/

Re: Query for JMX "Async Event Queue properties"

2017-07-10 Thread Barry Oglesby
Dinesh, The getRegionAssociatedMembers method looks like it could be improved. It currently does this: - gets the distributed MBean for the region - if one doesn't exist it returns null - if one does exist, it gets the members for that bean (which are the members with that region) - if there are

slf4j-jdk

2017-07-10 Thread Kirk Lund
Does anyone know how and where "slf4j-jdk" was added as a dependency and bundled jar? I get the feeling that everyone is expecting someone else (the person who added it?) is going to fix it. Was it added in a PR maybe? Error java.lang.AssertionError: The bundled jars have changed. Please make sur

Build failed in Jenkins: Geode-nightly #892

2017-07-10 Thread Apache Jenkins Server
See -- [...truncated 112.76 KB...] :geode-json:assemble :geode-json:compileTestJava UP-TO-DATE :geode-json:processTestResources :geode-json:testClasses :geode-json:checkMissedTests UP-TO-DATE

Re: getSizeInBytes() return type

2017-07-10 Thread Darrel Schneider
I agree with Jake that supporting values whose serialized size is larger than 2G would be a big change. Geode has a number of places in the implementation that use a byte array to store the serialized object form and the maximum array length in Java is 2G. On Mon, Jul 10, 2017 at 8:43 AM, Jacob Ba

Re: Review Request 60202: GEODE-3056: fix the message for invalid partition-resolver

2017-07-10 Thread Emily Yeh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60202/#review180043 --- Ship it! Ship It! - Emily Yeh On June 19, 2017, 7:08 p.m., J

Re: getSizeInBytes() return type

2017-07-10 Thread Jacob Barrett
Daniel, A change like this would be pretty deep and impact things like off heap and the network protocol. You really need to consider the problems with storing such large objects and ask yourself if you can do something different. Transferring a large object requires that it be serialized into me

[GitHub] geode pull request #611: GEODE-3145 Add new protocol to Geode JAR

2017-07-10 Thread pivotal-amurmann
Github user pivotal-amurmann closed the pull request at: https://github.com/apache/geode/pull/611 --- 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 featur