Re: Review Request 36397: HdfsStore API refactoring

2015-07-14 Thread Ashvin A
> On July 13, 2015, 2:37 p.m., Amogh Shetkar wrote: > > gemfire-core/src/main/java/com/gemstone/gemfire/cache/hdfs/HDFSStore.java, > > line 222 > > > > > > Minor suggestion: You may want to add explicitly that compa

Re: Review Request 36397: HdfsStore API refactoring

2015-07-14 Thread Ashvin A
> On July 10, 2015, 9:26 p.m., Dan Smith wrote: > > I have on minor comment - getPurgeInterval doesn't mention the time unit > > (seconds, milliseconds, etc.) in the javadocs. Otherwise, looks good! Thanks Dan. I have fixed the javadoc. - Ashvin -

Re: Review Request 36397: HdfsStore API refactoring

2015-07-14 Thread Ashvin A
> On July 13, 2015, 6:25 p.m., Darrel Schneider wrote: > > gemfire-core/src/main/java/com/gemstone/gemfire/cache/hdfs/HDFSStore.java, > > line 19 > > > > > > How is this disk store used in later steps? > > Is it

[GitHub] incubator-geode pull request: GEODE-21. Website updates from Proje...

2015-07-14 Thread jfarrell
Github user jfarrell closed the pull request at: https://github.com/apache/incubator-geode/pull/5 --- 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

[GitHub] incubator-geode pull request: Minor grammatical fixes

2015-07-14 Thread jfarrell
Github user jfarrell closed the pull request at: https://github.com/apache/incubator-geode/pull/4 --- 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

Review Request 36491: GEODE-96: fix race condition in unit test testDistributedRegionClientPutRejection

2015-07-14 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36491/ --- Review request for geode and Kirk Lund. Bugs: GEODE-96 https://issues.apach

Build failed in Jenkins: Geode-nightly #128

2015-07-14 Thread Apache Jenkins Server
See Changes: [klund] GEODE-94: Add typecast to prevent ambiguous method calls under JDK 8 -- [...truncated 180 lines...] ^

[GitHub] incubator-geode pull request: GEODE-115 Change gfsh path for integ...

2015-07-14 Thread jchen21
GitHub user jchen21 opened a pull request: https://github.com/apache/incubator-geode/pull/11 GEODE-115 Change gfsh path for integration tests Spark Geode Connector now uses gfsh from current Geode build. You can merge this pull request into a Git repository by running: $ git pu

Re: Apache Geode & Spring Data GemFire...

2015-07-14 Thread John Blum
Hi Dan- Generally, I monitor *Spring Data GemFire* builds here... https://build.spring.io/browse/SGF. However, given this is not as convenient as receiving an automatic notification, I setup an email notification in the Nightly-ApacheGeode build sending to the dev@geode list after "All Builds Com

Re: Apache Geode & Spring Data GemFire...

2015-07-14 Thread Dan Smith
Hi John, Sounds great! What's the best way to monitor this nightly build? Is there a way to get email alerts on build failures? -Dan On Tue, Jul 14, 2015 at 12:28 PM, John Blum wrote: > Hello Geode devs- > > It has always been Spring's philosophy to provide early, unprecedented > integration a

[GitHub] incubator-geode pull request: GEODE-83 Add support for Redis AUTH ...

2015-07-14 Thread vitogav
Github user vitogav commented on the pull request: https://github.com/apache/incubator-geode/pull/10#issuecomment-121380402 Done --- 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 e

Review Request 36484: GEODE-105: Null value in Map causes NPE with Map Indexes

2015-07-14 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36484/ --- Review request for geode, anilkumar gingade, Bruce Schuchardt, Jianxia Chen, Lyn

Review Request 36483: GEODE-91 Zero results when querying against a HashIndex indexed on PdxStrings

2015-07-14 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36483/ --- Review request for geode, anilkumar gingade, Bruce Schuchardt, Jianxia Chen, Lyn

Apache Geode & Spring Data GemFire...

2015-07-14 Thread John Blum
Hello Geode devs- It has always been Spring's philosophy to provide early, unprecedented integration and support for the latest Java enterprise technologies in order to reduce the complexity and boilerplate cruft that still plagues Java enterprise application development today. Spring continues t

Re: Review Request 36414: GEODE-74: Making the satisfy redundancy phase of rebalance parallel

2015-07-14 Thread Dan Smith
> On July 13, 2015, 4:20 p.m., Kirk Lund wrote: > > gemfire-core/src/test/java/com/gemstone/gemfire/internal/cache/partitioned/PartitionedRegionLoadModelJUnitTest.java, > > line 52 > > > > > > This looks more like an

[GitHub] incubator-geode pull request: GEODE-83 Add support for Redis AUTH ...

2015-07-14 Thread sbawaska
Github user sbawaska commented on the pull request: https://github.com/apache/incubator-geode/pull/10#issuecomment-121326508 Hi Vito, Can you please add unit tests for this pull request? --- If your project is set up for it, you can reply to this email and have your reply appear o

[GitHub] incubator-geode pull request: GEODE-83 Add support for Redis AUTH ...

2015-07-14 Thread markito
Github user markito commented on the pull request: https://github.com/apache/incubator-geode/pull/10#issuecomment-121317997 +1 --- 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 ena

Re: Apache Jenkins question

2015-07-14 Thread William Markito
It looks like this is related to the OOM problem while trying to collect and generate test results. On Tue, Jul 14, 2015 at 9:53 AM, Kirk Lund wrote: > Does anyone know why the Apache Jenkins does not show details on test > failures? > > Our internal Jenkins has a "Test Result" link with links

Re: Request to post Geode security article

2015-07-14 Thread Gregory Chase
Hi Guillermo, I have given you admin access onto https://github.com/Pivotal-Open-Source-Hub. Please transfer your sample project repository to this Github. Thanks! -Greg On Tue, Jul 14, 2015 at 9:36 AM, Gregory Chase wrote: > As I understand, someone has to post on your behalf now who has acc

Re: Nightly build test failures

2015-07-14 Thread Kirk Lund
Yep, #127 completed after I'd typed up my email and filed several tickets. #127 passed completely. Looks like #126 ran out of memory while gathering test results for reports so I'll file a ticket on that. I'll also individually review and test the distributed tests that failed in #126 before closin

Apache Jenkins question

2015-07-14 Thread Kirk Lund
Does anyone know why the Apache Jenkins does not show details on test failures? Our internal Jenkins has a "Test Result" link with links for each test failure that you can follow to see detailed info on the test. Apache Jenkins doesn't have this and the only way I can see results is in "Console Ou

Re: Nightly build test failures

2015-07-14 Thread Ashvin A
Seems builds fails on ubuntu hosts :-? Build ↑ Duration Slave

Re: Nightly build test failures

2015-07-14 Thread Anthony Baker
Looks like #127 didn’t have any failures. Changes GEODE-16 [DISTTX] Bringing pending changes for distributed transactions (still work in progress) from one of the internal gemfire branch (develop_dist_tx) to ASF i.e. (detail

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

2015-07-14 Thread Apache Jenkins Server
See

Re: Request to post Geode security article

2015-07-14 Thread Gregory Chase
As I understand, someone has to post on your behalf now who has access to the blog. I think I am one such somebody, so let me see how to post it for you :) Is there a particular ID you would like me to attribute it to for someone who wants to contact you? Twitter for Github or just an email? On

Re: Review Request 36416: GEODE-56 Decrementing bytesOverflowedOnDisk when an update happens to an evicted entry.

2015-07-14 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36416/#review91628 --- Ship it! Ship It! - Darrel Schneider On July 13, 2015, 6:03 p.m.

Re: Review Request 36414: GEODE-74: Making the satisfy redundancy phase of rebalance parallel

2015-07-14 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36414/#review91627 --- Ship it! Ship It! - Darrel Schneider On July 13, 2015, 6:13 p.m.

Nightly build test failures

2015-07-14 Thread Kirk Lund
Looks like we have nightly build failures. I think this means we need to freeze all checkins and work to stabilize these test failures: https://builds.apache.org/view/All/job/Geode-nightly/lastFailedBuild/console Jenkins also ran out of memory trying to archive test results. Any ideas what to do

Re: Review Request 36408: Resolve the Geode Spark Connector Build Issue

2015-07-14 Thread Qihong Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36408/#review91625 --- Ship it! Ship It! - Qihong Chen On July 10, 2015, 11:32 p.m., Ji

Re: Request to post Geode security article

2015-07-14 Thread gtantachuco .
Thanks guys for your input. I made the changes you suggested and also made Geode sample's Github repo public: https://github.com/gtantachuco-pivotal/geode-security-samples What would the next steps be? On Sat, Jul 11, 2015 at 12:07 PM, Jags Ramnarayanan wrote: > Guillermo, >Glad to see you a

[GitHub] incubator-geode pull request: GEODE-83 Add support for Redis AUTH ...

2015-07-14 Thread vitogav
GitHub user vitogav opened a pull request: https://github.com/apache/incubator-geode/pull/10 GEODE-83 Add support for Redis AUTH command Also added Geode property "redis-password" so the authentication password can be set by xml or through GFSH You can merge this pull request into

Review Request 36477: GEODE-100: Nested count(*) query on pr causes NullPointerException

2015-07-14 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36477/ --- Review request for geode, anilkumar gingade, Bruce Schuchardt, Jianxia Chen, Lyn