Fixed: apache/geode#3963 (develop - bbfdb06)

2017-09-22 Thread Travis CI
Build Update for apache/geode - Build: #3963 Status: Fixed Duration: 12 minutes and 20 seconds Commit: bbfdb06 (develop) Author: Dan Smith Message: Adding a new framework for concurrency testing in geode (#787) * Adding a new framework for concurrency testing

Broken: apache/geode#3959 (develop - 2cb5f92)

2017-09-22 Thread Travis CI
Build Update for apache/geode - Build: #3959 Status: Broken Duration: 13 minutes and 37 seconds Commit: 2cb5f92 (develop) Author: Jared Stewart Message: GEODE-3618: Move GfshRule into geode-junit This closes #781. View the changeset: https://github.com/apach

Fixed: apache/geode#3962 (develop - 1779a94)

2017-09-22 Thread Travis CI
Build Update for apache/geode - Build: #3962 Status: Fixed Duration: 13 minutes and 48 seconds Commit: 1779a94 (develop) Author: Jared Stewart Message: Fix compilation failures from rebase conflict View the changeset: https://github.com/apache/geode/compare/1

Broken: apache/geode#3961 (develop - 1de8b77)

2017-09-22 Thread Travis CI
Build Update for apache/geode - Build: #3961 Status: Broken Duration: 13 minutes and 11 seconds Commit: 1de8b77 (develop) Author: Patrick Rhomberg Message: GEODE-3539: Add missing test coverage for 'describe config' command. This closes #800. View the changes

Broken: apache/geode-native#436 (develop - aa997cd)

2017-09-22 Thread Travis CI
Build Update for apache/geode-native - Build: #436 Status: Broken Duration: 12 minutes and 41 seconds Commit: aa997cd (develop) Author: Jacob Barrett Message: GEODE-3630: Change to void Serializable::fromData(...) - Change .NET to void Serializable::FromData(

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

2017-09-22 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #687 was successful. --- Scheduled 2044 tests in total. https://build.spring.io/browse/SGF-NAG-687/ -- This

Re: []DISCUSS] Using package names to identify public API's

2017-09-22 Thread Brian Baynes
Yep, GEODE-3473 will make it in -- have been waiting for a good moment when there aren't other items in flight that would need to be rebased. On Fri, Sep 22, 2017 at 2:31 PM, Dan Smith wrote: > I found bug GEODE-3473 which was created based on this discussion. I marked > that as requiring a fix

Re: Please don't commit broken javadocs

2017-09-22 Thread Galen O'Sullivan
We'll move everything to internal soon, before public release. We have a ticket to fix the Javadocs and the links will be removed soon as well. The broken links are for generated files that don't seem to have been created when the javadoc task runs. On Fri, Sep 22, 2017 at 2:27 PM, Dan Smith wro

Re: []DISCUSS] Using package names to identify public API's

2017-09-22 Thread Dan Smith
I found bug GEODE-3473 which was created based on this discussion. I marked that as requiring a fix for 1.3.0, so we don't ship these protobuf related internal classes as part of the public API. -Dan On Sat, Aug 12, 2017 at 9:42 AM, John Blum wrote: > On the flip side of internal vs. external A

Re: Please don't commit broken javadocs

2017-09-22 Thread Dan Smith
Regarding the duplicate javadocs from geode-assembly - geode-assembly builds the public javadocs for all of geode. Each module also builds it's own javadocs which includes all of the internal methods. That does point out that everything in geode-protobuf seems to be in non internal packages, meani

Please don't commit broken javadocs

2017-09-22 Thread Kirk Lund
I'm seeing newly broken javadocs being committed. :geode-core:javadoc /Users/klund/dev/gemfire/open/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/MessageExecutionContext.java:91: warning - @return tag has no arguments. 1 warning :geode-protobuf:javadoc/tmp/build/ae3c03f4/g

Re: Review Request 62506: GEODE-3679: Original client member Id in the transaction should be set in the PartitionMessage

2017-09-22 Thread Eric Shu
> On Sept. 22, 2017, 6:55 p.m., anilkumar gingade wrote: > > geode-core/src/test/java/org/apache/geode/internal/cache/ClientServerTransactionDUnitTest.java > > Lines 4111 (patched) > > > > > > Do we need to check fo

Re: Review Request 62506: GEODE-3679: Original client member Id in the transaction should be set in the PartitionMessage

2017-09-22 Thread Eric Shu
> On Sept. 22, 2017, 6:27 p.m., Nick Reich wrote: > > geode-core/src/test/java/org/apache/geode/internal/cache/ClientServerTransactionDUnitTest.java > > Lines 4109 (patched) > > > > > > This is actually "doTransacti

Re: Review Request 62506: GEODE-3679: Original client member Id in the transaction should be set in the PartitionMessage

2017-09-22 Thread Eric Shu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62506/ --- (Updated Sept. 22, 2017, 7:12 p.m.) Review request for geode, anilkumar gingade

Re: Review Request 62506: GEODE-3679: Original client member Id in the transaction should be set in the PartitionMessage

2017-09-22 Thread anilkumar gingade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62506/#review186005 --- geode-core/src/test/java/org/apache/geode/internal/cache/ClientSe

Re: Review Request 62506: GEODE-3679: Original client member Id in the transaction should be set in the PartitionMessage

2017-09-22 Thread Nick Reich
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62506/#review186003 --- geode-core/src/test/java/org/apache/geode/internal/cache/ClientSe

Review Request 62506: GEODE-3679: Original client member Id in the transaction should be set in the PartitionMessage

2017-09-22 Thread Eric Shu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62506/ --- Review request for geode, anilkumar gingade, Darrel Schneider, Kirk Lund, Lynn G

Minor dunit cleanup being merged to develop

2017-09-22 Thread Kirk Lund
GEODE-3638: cleanup minor issues found while prepping dunit talk My precheckin for PR #802 (GEODE-3638) is GREEN except for *org.apache.geode.internal.cache.tier.sockets.ClientHealthMonitorJUnitTest* which is still failing. #802 introduces DistributedTestRule and fixes DistributedDisconnectRule a

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

2017-09-22 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #686 was successful. --- This was manually triggered by John Blum. 2044 tests in total. https://build.spring.io/b

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

2017-09-22 Thread Apache Jenkins Server
See Changes: [abaker] Update dockerfile for 1.2.1 [github] GEODE-2788 Document defn of pool attr socket-connect-timeout (#798) [klund] GEODE-3205: add FlakyTest category [klund] GEODE-3552: delete TestSuite

Build failed in Jenkins: Geode-nightly #962

2017-09-22 Thread Apache Jenkins Server
See Changes: [abaker] Update dockerfile for 1.2.1 [github] GEODE-2788 Document defn of pool attr socket-connect-timeout (#798) [klund] GEODE-3205: add FlakyTest category [klund] GEODE-3552: delete TestSuite that d

Re: Need Help on understanding vsd files

2017-09-22 Thread Avinash Dongre
Hi Barry, Still cannot see them. Thanks Avinash On Fri, Sep 22, 2017 at 2:18 PM, Barry Oglesby wrote: > Interesting. I can see them, but here they are again. I dragged/dropped > them this time. > > [image: Inline image 1][image: Inline image 2][image: Inline image 3][image: > Inline image 4][i

Correct Geode REST API cipher

2017-09-22 Thread Rupert St John Webster
Hi, Please can someone advise which cipher to use for ssl-enabled-components=web and ssl-ciphers=any After starting Geode with web security then when we run up Swagger using https all we see is ERR_SSL_VERSION_OR_CIPHER_MISMATCH I also asked on StackOverflow https://stackoverflow.com/question

Re: Need Help on understanding vsd files

2017-09-22 Thread Barry Oglesby
Interesting. I can see them, but here they are again. I dragged/dropped them this time. [image: Inline image 1][image: Inline image 2][image: Inline image 3][image: Inline image 4][image: Inline image 5] Thanks, Barry Oglesby On Thu, Sep 21, 2017 at 10:23 PM, Avinash Dongre wrote: > Thanks Ba