[GitHub] incubator-geode issue #177: GEODE-11: Index and AEQ stored in the same disk ...

2016-06-30 Thread gesterzhou
Github user gesterzhou commented on the issue: https://github.com/apache/incubator-geode/pull/177 +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 enabled and wishes so, or if

[GitHub] incubator-geode issue #183: GEODE-1615: gfsh unable to destroy region that h...

2016-06-30 Thread jinmeiliao
Github user jinmeiliao commented on the issue: https://github.com/apache/incubator-geode/pull/183 the fix looks good. Where is the test? --- 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

Geode website tagged with minor Apache branding issue in audit

2016-06-30 Thread Greg Chase
Dear Geode community, I see the Apache Geode website got tagged with a minor branding concern in a recent audit of incubating projects. See the wiki entry here: https://wiki.apache.org/incubator/BrandingAuditJune2016 See start of the thread in gene...@incubator.apache.org here:

Re: Valid characters in region names?

2016-06-30 Thread William Markito
We already discussed this on the previous thread and got general consensus on getting "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz12345678 90_-" for Geode... Anything other than this can be problematic for JMX and other components that will then require special treatment to handle

Re: update website for WAN, CQ and native client

2016-06-30 Thread Swapnil Bawaskar
Thanks everyone for chiming in! These change have been pushed. checkout: http://geode.incubator.apache.org/ On Wed, Jun 22, 2016 at 11:34 AM, Udo Kohlmeyer wrote: > +1, > > to me "Multi-site" is too rigid. People might forget that this is a > inter-cluster comms channel.

[GitHub] incubator-geode pull request #177: GEODE-11: Index and AEQ stored in the sam...

2016-06-30 Thread nabarunnag
Github user nabarunnag commented on a diff in the pull request: https://github.com/apache/incubator-geode/pull/177#discussion_r69231478 --- Diff: geode-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/LuceneIndexForPartitionedRegion.java --- @@ -77,13 +77,13 @@

[GitHub] incubator-geode issue #177: GEODE-11: Index and AEQ stored in the same disk ...

2016-06-30 Thread nabarunnag
Github user nabarunnag commented on the issue: https://github.com/apache/incubator-geode/pull/177 +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 enabled and wishes so, or if

Review Request 49481: summarize the investigation results on numeric field range query into a integration test case

2016-06-30 Thread xiaojian zhou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49481/ --- Review request for geode, anilkumar gingade and Dan Smith. Bugs: GEODE-11

Re: Valid characters in region names?

2016-06-30 Thread Anilkumar Gingade
Here is what GemFire docs states about region names: http://gemfire.docs.pivotal.io/docs-gemfire/latest/basic_config/data_regions/region_naming.html But again, we have supported other chars as part of region names... The Query engine allows/detects region name with: "_", “+”, “-“, “:”, “#”,

Re: Review Request 49102: WAN Ack reader thread needs to be shut down before sending a close connection

2016-06-30 Thread nabarun nag
> On June 30, 2016, 11:14 p.m., nabarun nag wrote: > > geode-wan/src/main/java/com/gemstone/gemfire/internal/cache/wan/GatewaySenderEventRemoteDispatcher.java, > > line 303 > > > > > > Hi Jason, > > While

Re: Review Request 49102: WAN Ack reader thread needs to be shut down before sending a close connection

2016-06-30 Thread nabarun nag
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49102/#review140264 ---

[GitHub] incubator-geode pull request #184: GEODE-1452 Add @Ignore to disabled tests

2016-06-30 Thread gracemeilen
GitHub user gracemeilen opened a pull request: https://github.com/apache/incubator-geode/pull/184 GEODE-1452 Add @Ignore to disabled tests You can merge this pull request into a Git repository by running: $ git pull https://github.com/gracemeilen/incubator-geode

Re: Review Request 49471: GEODE-1374: Flaky tests will now have junit xml recorded into seperate directory

2016-06-30 Thread nabarun nag
> On June 30, 2016, 11:06 p.m., nabarun nag wrote: > > Ship It! Change made in Jenkins Precheckin to pick up the new folder. - nabarun --- This is an automatically generated e-mail. To reply, visit:

Re: Review Request 49471: GEODE-1374: Flaky tests will now have junit xml recorded into seperate directory

2016-06-30 Thread nabarun nag
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49471/#review140261 --- Ship it! Ship It! - nabarun nag On June 30, 2016, 8:50

Re: [INFO REQUEST] Uploading the podling report

2016-06-30 Thread Nabarun Nag
Thanks Swapnil. Email sent. On Thu, Jun 30, 2016 at 3:51 PM Swapnil Bawaskar wrote: > Send an email to gene...@incubator.apache.org asking for permissions to > the > wiki for uploading the podling report. > > On Thu, Jun 30, 2016 at 3:19 PM, Nabarun Nag

Re: [INFO REQUEST] Uploading the podling report

2016-06-30 Thread Swapnil Bawaskar
Send an email to gene...@incubator.apache.org asking for permissions to the wiki for uploading the podling report. On Thu, Jun 30, 2016 at 3:19 PM, Nabarun Nag wrote: > Hi, > > Thank you all for the reviews. > > Can someone please tell me the steps to append the podling report

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

2016-06-30 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #356 was successful. --- Scheduled 1412 tests in total. https://build.spring.io/browse/SGF-NAG-356/ -- This

Re: Valid characters in region names?

2016-06-30 Thread Karen Miller
Here is the page of the documentation on region naming: http://docs-geode-develop.cfapps.io/docs/basic_config/data_regions/region_naming.html I'm guessing that a JIRA for also including a dash was not implemented. When you find the JIRA number, please post it on this thread. On Thu, Jun 30,

[INFO REQUEST] Uploading the podling report

2016-06-30 Thread Nabarun Nag
Hi, Thank you all for the reviews. Can someone please tell me the steps to append the podling report to http://wiki.apache.org/incubator/July2016. Regards Nabarun Nag

[GitHub] incubator-geode pull request #183: GEODE-1615: gfsh unable to destroy region...

2016-06-30 Thread kjduling
GitHub user kjduling opened a pull request: https://github.com/apache/incubator-geode/pull/183 GEODE-1615: gfsh unable to destroy region that has an hyphen in its name Added a test for whether or not the region contains a hyphen in the name. If it does, it is necessary to wrap the

[GitHub] incubator-geode pull request #182: GEODE-1566: rename GeodeRedisServer and r...

2016-06-30 Thread gracemeilen
GitHub user gracemeilen opened a pull request: https://github.com/apache/incubator-geode/pull/182 GEODE-1566: rename GeodeRedisServer and repackage in org.apache.geode You can merge this pull request into a Git repository by running: $ git pull

Review Request 49471: GEODE-1374: Flaky tests will now have junit xml recorded into seperate directory

2016-06-30 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49471/ --- Review request for geode, Kirk Lund, nabarun nag, and Dan Smith. Repository:

Re: Valid characters in region names?

2016-06-30 Thread Kevin Duling
Do you know if a Jira ticket was opened on it? I'm currently working on https://issues.apache.org/jira/browse/GEODE-1615 and would like to link to it. On Thu, Jun 30, 2016 at 12:38 PM, Michael Stolz wrote: > Yes we did discuss adding hyphen in Geode, and we discussed

Re: Valid characters in region names?

2016-06-30 Thread Michael Stolz
Yes we did discuss adding hyphen in Geode, and we discussed actually checking during create. -- Mike Stolz Principal Engineer, GemFire Product Manager Mobile: 631-835-4771 On Thu, Jun 30, 2016 at 3:30 PM, Kevin Duling wrote: > I found a thread from March of this year where

Re: Valid characters in region names?

2016-06-30 Thread Kevin Duling
I found a thread from March of this year where this was discussed and '-' is also included in that list. Shouldn't the creation of a region with an invalid character raise an exception? I'm finding that I can create and list regions with a variety of non-alphanumeric characters. But I can only

Re: Review Request 49102: WAN Ack reader thread needs to be shut down before sending a close connection

2016-06-30 Thread Jason Huynh
> On June 29, 2016, 9:02 p.m., anilkumar gingade wrote: > > Nice to have unit test for this. Agreed, however this would require quite a few test hooks in product code. There are multiple scenarios that can cause this issue. I've inserted a "test case" to the ticket that will reproduce the

[GitHub] incubator-geode pull request #181: GEODE-1598 Fixed autocomplete for first o...

2016-06-30 Thread gracemeilen
GitHub user gracemeilen opened a pull request: https://github.com/apache/incubator-geode/pull/181 GEODE-1598 Fixed autocomplete for first option You can merge this pull request into a Git repository by running: $ git pull https://github.com/gracemeilen/incubator-geode

[GitHub] incubator-geode pull request #180: GEODE-1598 Fixed autocomplete for first o...

2016-06-30 Thread gracemeilen
Github user gracemeilen closed the pull request at: https://github.com/apache/incubator-geode/pull/180 --- 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

Build failed in Jenkins: Geode-nightly #515

2016-06-30 Thread Apache Jenkins Server
See Changes: [klund] GEODE-1598: avoid deletion of options during GFSH autocomplete [upthewaterspout] GEODE-11: Cleaning up old IndexRepositoryImpls when buckets move -- [...truncated 536