[GitHub] geode pull request #606: GEODE-2601: Fixing startup configurations logging t...

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

[GitHub] geode pull request #630: GEODE-3141: GetRegion Operation implemented

2017-07-18 Thread kohlmu-pivotal
Github user kohlmu-pivotal commented on a diff in the pull request: https://github.com/apache/geode/pull/630#discussion_r128038401 --- Diff: geode-protobuf/src/main/proto/basicTypes.proto --- @@ -52,7 +52,12 @@ message CallbackArguments { message Region { --- End diff

[GitHub] geode pull request #630: GEODE-3141: GetRegion Operation implemented

2017-07-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/630 --- 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: Review Request 60935: GEODE-3115 Added changes to check for persistent region during pdx type registry.

2017-07-18 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60935/#review180828 --- Fix it, then Ship it! fix then ship geode-core/src/main/java/

Re: Review Request 60935: GEODE-3115 Added changes to check for persistent region during pdx type registry.

2017-07-18 Thread Nick Reich
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60935/#review180818 --- geode-core/src/test/java/org/apache/geode/pdx/PdxAttributesJUnitT

Re: Proposal: Lucene indexing/searching for nested objects

2017-07-18 Thread Dan Smith
I think this LuceneSerializer API needs a slight tweak. In order to implement the proposed FlatFormatSerializer, the serializer needs access to the index configuration to see what fields the user wants to index. We should also pass the LuceneIndex to the serializer. public interface LuceneSerializ

[GitHub] geode pull request #634: Feature/geode 3175

2017-07-18 Thread galen-pivotal
Github user galen-pivotal commented on a diff in the pull request: https://github.com/apache/geode/pull/634#discussion_r128082009 --- Diff: geode-core/src/test/java/org/apache/geode/test/dunit/standalone/ChildVM.java --- @@ -49,8 +49,8 @@ public static void main(String[] args) thro

[GitHub] geode pull request #634: Feature/geode 3175

2017-07-18 Thread galen-pivotal
Github user galen-pivotal commented on a diff in the pull request: https://github.com/apache/geode/pull/634#discussion_r128082288 --- Diff: geode-core/src/test/java/org/apache/geode/test/dunit/standalone/ProcessManager.java --- @@ -90,12 +92,17 @@ public synchronized void launchVM

[GitHub] geode issue #404: Geode 2469

2017-07-18 Thread metatype
Github user metatype commented on the issue: https://github.com/apache/geode/pull/404 What would it take to get this PR merged? --- 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 en

[GitHub] geode pull request #642: GEODE-3031: Extracting startLocator and startServer...

2017-07-18 Thread YehEmily
GitHub user YehEmily opened a pull request: https://github.com/apache/geode/pull/642 GEODE-3031: Extracting startLocator and startServer from LauncherLifecycleCommands [See the JIRA ticket here.](https://issues.apache.org/jira/browse/GEODE-3031?jql=project%20%3D%20GEODE%20AND%20tex

[Spring CI] Spring Data GemFire > Nightly-ApacheGeode > #620 has FAILED (1 tests failed)

2017-07-18 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #620 failed. --- Scheduled 1/1956 tests failed. https://build.spring.io/browse/SGF-NAG-620/

[GitHub] geode pull request #640: GEODE-3232: Get a snapshot of maps when serializing...

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

[GitHub] geode pull request #643: GEODE-3192,GEODE-3229: Change API and implementatio...

2017-07-18 Thread WireBaron
GitHub user WireBaron opened a pull request: https://github.com/apache/geode/pull/643 GEODE-3192,GEODE-3229: Change API and implementation of protobuf PutAll. @kohlmu-pivotal @pivotal-amurmann @galen-pivotal @bschuchardt @hiteshk25 * We will now dispatch incoming protobuf P

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

2017-07-18 Thread Apache Jenkins Server
See

SDG DiskStoreDirectoryBeanPostProcessorIntegrationTests failed

2017-07-18 Thread Kirk Lund
Looks like a Geode change probably broke DiskStoreDirectoryBeanPostProcessorIntegrationTests in Spring Data GemFire. I can't tell what actually broke it without opening the test in IntelliJ. https://build.spring.io/browse/SGF-NAG-620/ org.junit.ComparisonFailure: expected:<[tru]e> but was:<[fals]

RE: Stored procedures on Apache Geode.

2017-07-18 Thread marios390
Hi John, As mentioned the other day, we are interested in lift all the stored procedures up into the cluster.Do you think this indeed will have a huge impact on the performance? Is there any best practises for doing this or any demo you could provide demonstrating this process?(SP to geode/gemf