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

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...

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 f

Geode website tagged with minor Apache branding issue in audit

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: http://mail-archiv

Re: Valid characters in region names?

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 unusual

Re: update website for WAN, CQ and native client

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. Which would allow them

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

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 @@ publ

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

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

--- 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 h

Re: Valid characters in region names?

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

> 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 debug

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49102/#review140264 --- geode-wan/src/main/java/com/gemstone/gemfire/internal/cache/wan/G

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

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 feature/GEODE

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

> 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: https://reviews.apache.org/r/49471/#r

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

--- 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 p.m.,

Re: [INFO REQUEST] Uploading the podling report

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 wrote: > > > Hi, > > > > Thank you all fo

Re: [INFO REQUEST] Uploading the podling report

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 to > http://wiki.a

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

--- 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?

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, 201

[INFO REQUEST] Uploading the podling report

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...

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...

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 https://github.com/gracemeilen/i

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

--- 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: ge

Re: Valid characters in region names?

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 actually > checking during

Re: Valid characters in region names?

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 this was discussed an

Re: Valid characters in region names?

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 d

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

> 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 is

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

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 feature

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

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 f

Re: Valid characters in region names?

The only characters that should be used in Region names are ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz1234567890_ -- Mike Stolz Principal Engineer, GemFire Product Manager Mobile: 631-835-4771 On Thu, Jun 30, 2016 at 2:03 PM, Kevin Duling wrote: > I'm working on a bug where it is not

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

GitHub user gracemeilen opened a pull request: https://github.com/apache/incubator-geode/pull/180 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 feature

Valid characters in region names?

I'm working on a bug where it is not possible to delete a region that has a hyphen in it. E.g., the region was created with: create region --name=not-good --type=REPLICATE I've a solution done, but see this as an opportunity to add a test for other special characters. Is there a list of valid a

Re: [GitHub] incubator-geode issue #178: GEODE-11: Pagination of Lucene query results aft...

Thats rightIts hard to predict if there could be any entries, by the time hasNext() and next() is called...We need to capture this in the the docs... -Anil. On Wed, Jun 29, 2016 at 5:27 PM, upthewaterspout wrote: > Github user upthewaterspout commented on the issue: > > https://github.

Re: Podling Report Reminder - July 2016

Looks good... -Anil. On Thu, Jun 30, 2016 at 7:55 AM, Swapnil Bawaskar wrote: > +1 > > On Thu, Jun 30, 2016 at 7:43 AM, Michael Stolz wrote: > > > +1 Looks good > > > > -- > > Mike Stolz > > Principal Engineer - Gemfire Product Manager > > Mobile: 631-835-4771 > > On Jun 28, 2016 7:48 PM, "Nab

Re: Review Request 49409: GEODE-1573: Use snappy java implementation

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49409/#review140187 --- geode-core/src/main/java/com/gemstone/gemfire/compression/SnappyC

Re: Podling Report Reminder - July 2016

+1 On Thu, Jun 30, 2016 at 7:43 AM, Michael Stolz wrote: > +1 Looks good > > -- > Mike Stolz > Principal Engineer - Gemfire Product Manager > Mobile: 631-835-4771 > On Jun 28, 2016 7:48 PM, "Nabarun Nag" wrote: > > > Hi > > > > Please find below the draft of the podling report. > > > > Any feed

Re: Podling Report Reminder - July 2016

+1 Looks good -- Mike Stolz Principal Engineer - Gemfire Product Manager Mobile: 631-835-4771 On Jun 28, 2016 7:48 PM, "Nabarun Nag" wrote: > Hi > > Please find below the draft of the podling report. > > Any feedback is welcome. > > Regards > Naba > > > > Geode > > Geode is

Re: Podling Report Reminder - July 2016

+1 2016-06-30 21:07 GMT+08:00 Anthony Baker : > > +1 > > > On Jun 29, 2016, at 8:13 PM, William Markito > wrote: > > > > +1. LGTM > > > > Sent from my iPhone > > > >> On Jun 28, 2016, at 4:48 PM, Nabarun Nag wrote: > >> > >> Hi > >> > >> Please find below the draft of the podling report. > >>

Build failed in Jenkins: Geode-nightly #515

See Changes: [klund] GEODE-1598: avoid deletion of options during GFSH autocomplete [upthewaterspout] GEODE-11: Cleaning up old IndexRepositoryImpls when buckets move -- [...truncated 536 lines...

[GitHub] incubator-geode issue #179: Feature/geode 88 - adding Makefiles and code fix...

Github user pivotal-jbarrett commented on the issue: https://github.com/apache/incubator-geode/pull/179 See CMakeList.txt for production of Makefiles and other project files. All project build files are produced via CMake. ACE and all other libraries that have native client sp

Re: Podling Report Reminder - July 2016

+1 > On Jun 29, 2016, at 8:13 PM, William Markito wrote: > > +1. LGTM > > Sent from my iPhone > >> On Jun 28, 2016, at 4:48 PM, Nabarun Nag wrote: >> >> Hi >> >> Please find below the draft of the podling report. >> >> Any feedback is welcome. >> >> Regards >> Naba >> >> --

[GitHub] incubator-geode issue #179: Feature/geode 88 - adding Makefiles and code fix...

Github user galpal commented on the issue: https://github.com/apache/incubator-geode/pull/179 submitted this pull request by mistake.. I need to add some more changes and then I'll open a new one --- If your project is set up for it, you can reply to this email and have your reply ap

[GitHub] incubator-geode pull request #179: Feature/geode 88 - adding Makefiles and c...

Github user galpal closed the pull request at: https://github.com/apache/incubator-geode/pull/179 --- 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 #179: Feature/geode 88 - adding Makefiles and c...

GitHub user galpal opened a pull request: https://github.com/apache/incubator-geode/pull/179 Feature/geode 88 - adding Makefiles and code fixes for c++ client 1) adding 3 makefiles to allow compilation of c++ client 2) linking to ACE dynamically instead of statically 3) remov