[GitHub] geode-native pull request #11: GEODE-2486: Initialize OpenSSL for DEFAULT ci...

Github user echobravopapa commented on a diff in the pull request: https://github.com/apache/geode-native/pull/11#discussion_r101287739 --- Diff: src/cryptoimpl/SSLImpl.cpp --- @@ -52,25 +52,23 @@ SSLImpl::SSLImpl(ACE_SOCKET sock, const char *pubkeyfile, if (SSLImpl::s

[jira] [Commented] (GEODE-2486) SSL ciphers other than NULL not supported

[ https://issues.apache.org/jira/browse/GEODE-2486?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15867917#comment-15867917 ] ASF GitHub Bot commented on GEODE-2486: --- Github user echobravopapa commented on a di

PROXY and CACHING_PROXY regions on Client

GEODE-1887 was filed to make sure that the user experience while using Geode is similar to RDBMS and other data products out there. While reviewing the pull request I realized that we need to make other op

Fwd: GeodeRedisAdapter improvments/feedback

Hello Hitesh, The following is my feedback. *1. Redis Type String* I like the idea of creating a region upfront. If we are still using the convention that internal region names start with "__" , then I would suggest something like a region named "__RedisString region" *2. List Type* I propose

Re: PROXY and CACHING_PROXY regions on Client

I have strong fears that if we make these wholesale changes to existing APIs we're going to end up breaking lots of existing code. For instance, if we make destroyRegion propagate when it never did before, we may end up destroying a server side region in production that wasn't expected. I will ad

Build failed in Jenkins: Geode-nightly #749

See Changes: [kmiller] GEODE-2479 Remove docs reference to gemstone.com package [jiliao] GEODE-2474: mark NetstatDUnitTest as flaky [jiliao] refactor ServerStarterRule and LocatorStarterRule so that they can be [gzhou] GEODE-2471: fix t

Re: [DISCUSS] JIRA guidelines

+1 > On 14 Feb, 2017, at 20:50, William Markito Oliveira > wrote: > > +1 > > Finally!! ;) > > Sent from my iPhone > >> On Feb 14, 2017, at 7:59 PM, Galen M O'Sullivan >> wrote: >> >> +1 to the article and removing the draft label >> >>> On Tue, Feb 14, 2017 at 4:05 PM, Akihiro Kitada w

Re: PROXY and CACHING_PROXY regions on Client

I agree with Mike that whatever changes in behavior are made to the Java client library should also be made in the C++ and C# libraries. Sarge > On 15 Feb, 2017, at 08:02, Michael Stolz wrote: > > I have strong fears that if we make these wholesale changes to existing > APIs we're going to end

Re: PROXY and CACHING_PROXY regions on Client

I agree with what Mike said. Plus, I don't think this is as simple as it appears. For instance, if I ... Region students = clientCache.getRegion("Students"); What happens when the "Students" Region does not exist server-side? This would require a check to determine whether the Students Region

Re: Build failed in Jenkins: Geode-nightly #749

I don't seem to have access to see the build report. Is that restricted to committers only? Thanks, Galen On Wed, Feb 15, 2017 at 8:08 AM, Apache Jenkins Server < jenk...@builds.apache.org> wrote: > See > > Changes: > > [kmiller] GEODE-24

Re: PROXY and CACHING_PROXY regions on Client

John, The proposed solution is actually try to solve the situation you mentioned -- create a proxy Region failed silently on a client when region does not reside on the server. Region students = clientCache.getRegion("Students"); The getRegion will check on the server side and it should fail wit

Re: PROXY and CACHING_PROXY regions on Client

Introducing an API like this gives us the opportunity to split the client/server region API’s. I don’t think we should return Region, but something specific to “server view”. How would those API’s operate on a CACHING_PROXY? Anthony > On Feb 15, 2017, at 6:44 AM, Swapnil Bawaskar wrote: >

[jira] [Resolved] (GEODE-2442) Address link breaking

[ https://issues.apache.org/jira/browse/GEODE-2442?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ernest Burghardt resolved GEODE-2442. - Resolution: Invalid > Address link breaking > - > > Ke

Re: Review Request 56633: GEODE-2474: refactor code to use SystemUtils to read OS system props

On Feb. 14, 2017, 4:33 p.m., Kirk Lund wrote: > > Everything else looks great. It happens from following TDD. I write the test first and then write the code. This is an example where the results of TDD produce a test with questionable value, but if the tests are very fast (and they are) then i

Re: PROXY and CACHING_PROXY regions on Client

@Eric- Hmm... Well, I'd argue that it is still confusing to "*overload*" the purpose of getRegion("path") to dually "*get*" (the primary function/purpose) and also "*create*" (secondary). I'd also say that the getRegion("path") API call is not exclusive to a *ClientCache*, particularly since get

Re: Review Request 56668: GEODE-2474: mark NetstatDUnitTest as flaky

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56668/#review165730 --- Ship it! Ship It! - Kevin Duling On Feb. 14, 2017, 9:01 a.m.

Re: Review Request 56637: refactor ServerStarterRule and LocatorStarterRule so that they can be created without a Properties first.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56637/#review165731 --- Ship it! Ship It! - Kevin Duling On Feb. 14, 2017, 11 a.m.,

Re: Review Request 56635: GEODE-2481: extract default properties generation to its own class

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56635/#review165732 --- Ship it! Ship It! - Kevin Duling On Feb. 13, 2017, 6:33 p.m.

Re: Review Request 56682: GEODE-2444: move Redis out of geode-core.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56682/ --- (Updated Feb. 15, 2017, 5:53 p.m.) Review request for geode, Bruce Schuchardt,

[GitHub] geode pull request #398: Split the redis adapter into its own package

GitHub user galen-pivotal opened a pull request: https://github.com/apache/geode/pull/398 Split the redis adapter into its own package Under this PR, the redis adapter is moved to its own source root, and registered as a service. We're intending to make this a feature branc

[jira] [Commented] (GEODE-2421) Create VS2015 AMI

[ https://issues.apache.org/jira/browse/GEODE-2421?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868277#comment-15868277 ] Michael Martell commented on GEODE-2421: Since VS 2017 is needed by dodNetty, Erni

[jira] [Created] (GEODE-2488) NetstatDUnitTest fails with OutOfMemoryError

Kirk Lund created GEODE-2488: Summary: NetstatDUnitTest fails with OutOfMemoryError Key: GEODE-2488 URL: https://issues.apache.org/jira/browse/GEODE-2488 Project: Geode Issue Type: Bug

[jira] [Updated] (GEODE-2488) NetstatDUnitTest fails with OutOfMemoryError

[ https://issues.apache.org/jira/browse/GEODE-2488?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kirk Lund updated GEODE-2488: - Labels: MiscellaneousCommands NetstatCommand netstat (was: ) > NetstatDUnitTest fails with OutOfMemoryErr

[jira] [Updated] (GEODE-2488) NetstatDUnitTest fails with OutOfMemoryError

[ https://issues.apache.org/jira/browse/GEODE-2488?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kirk Lund updated GEODE-2488: - Component/s: management > NetstatDUnitTest fails with OutOfMemoryError > -

[jira] [Commented] (GEODE-2485) CacheTransactionManager suspend/resume can leak memory for 30 minutes

[ https://issues.apache.org/jira/browse/GEODE-2485?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868312#comment-15868312 ] Eric Shu commented on GEODE-2485: - Suspend and resume could be called from product. For e

[jira] [Commented] (GEODE-2488) NetstatDUnitTest fails with OutOfMemoryError

[ https://issues.apache.org/jira/browse/GEODE-2488?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868317#comment-15868317 ] Kirk Lund commented on GEODE-2488: -- NetstatFunction reads in every line from the netstat

[jira] [Comment Edited] (GEODE-2488) NetstatDUnitTest fails with OutOfMemoryError

[ https://issues.apache.org/jira/browse/GEODE-2488?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868317#comment-15868317 ] Kirk Lund edited comment on GEODE-2488 at 2/15/17 6:37 PM: --- Nets

[jira] [Updated] (GEODE-2421) Create VS2017 AMI

[ https://issues.apache.org/jira/browse/GEODE-2421?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ernest Burghardt updated GEODE-2421: Description: VS2017 is due to RTM in early March 2017, it would be nice to have a dev AMI fo

[GitHub] geode-native pull request #11: GEODE-2486: Initialize OpenSSL for DEFAULT ci...

Github user asfgit closed the pull request at: https://github.com/apache/geode-native/pull/11 --- 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

[jira] [Comment Edited] (GEODE-2421) Create VS2017 AMI

[ https://issues.apache.org/jira/browse/GEODE-2421?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868277#comment-15868277 ] Michael Martell edited comment on GEODE-2421 at 2/15/17 6:45 PM: ---

[jira] [Commented] (GEODE-2486) SSL ciphers other than NULL not supported

[ https://issues.apache.org/jira/browse/GEODE-2486?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868328#comment-15868328 ] ASF GitHub Bot commented on GEODE-2486: --- Github user asfgit closed the pull request

[jira] [Commented] (GEODE-2486) SSL ciphers other than NULL not supported

[ https://issues.apache.org/jira/browse/GEODE-2486?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868327#comment-15868327 ] ASF subversion and git services commented on GEODE-2486: Commit ad

[GitHub] geode pull request #398: Split the redis adapter into its own package

Github user scmbuildguy commented on a diff in the pull request: https://github.com/apache/geode/pull/398#discussion_r101350393 --- Diff: geode-core/src/main/java/org/apache/geode/redis/GeodeRedisService.java --- @@ -0,0 +1,33 @@ +/* + * Licensed to the Apache Software Foun

[GitHub] geode pull request #398: Split the redis adapter into its own package

Github user scmbuildguy commented on a diff in the pull request: https://github.com/apache/geode/pull/398#discussion_r101351743 --- Diff: geode-redis/src/test/java/org/apache/geode/redis/AuthJUnitTest.java --- @@ -0,0 +1,120 @@ +/* + * Licensed to the Apache Software Founda

[GitHub] geode-native pull request #3: Replace ace calls to standard functions

Github user asfgit closed the pull request at: https://github.com/apache/geode-native/pull/3 --- 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

[jira] [Commented] (GEODE-2484) Remove ACE from native client dependencies

[ https://issues.apache.org/jira/browse/GEODE-2484?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868337#comment-15868337 ] ASF subversion and git services commented on GEODE-2484: Commit d4

[jira] [Commented] (GEODE-2485) CacheTransactionManager suspend/resume can leak memory for 30 minutes

[ https://issues.apache.org/jira/browse/GEODE-2485?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868344#comment-15868344 ] Eric Shu commented on GEODE-2485: - Stack trace for the above scenario: {noformat}

[GitHub] geode pull request #398: Split the redis adapter into its own package

Github user galen-pivotal commented on a diff in the pull request: https://github.com/apache/geode/pull/398#discussion_r101354013 --- Diff: geode-core/src/main/java/org/apache/geode/redis/GeodeRedisService.java --- @@ -0,0 +1,33 @@ +/* + * Licensed to the Apache Software Fo

[jira] [Commented] (GEODE-2475) Upgrade lucene version to 6.4.1

[ https://issues.apache.org/jira/browse/GEODE-2475?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868347#comment-15868347 ] ASF subversion and git services commented on GEODE-2475: Commit 5d

[jira] [Commented] (GEODE-2398) Sporadic Oplog corruption due to channel.write failure

[ https://issues.apache.org/jira/browse/GEODE-2398?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868348#comment-15868348 ] ASF subversion and git services commented on GEODE-2398: Commit 9b

[jira] [Commented] (GEODE-2449) Move redis adapter to extension framework

[ https://issues.apache.org/jira/browse/GEODE-2449?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868351#comment-15868351 ] ASF subversion and git services commented on GEODE-2449: Commit f7

Re: Build failed in Jenkins: Geode-nightly #749

Hi Galen, I am able to the console log [1], test report summary [2], and artifacts [3] without logging in. Which 'build report' are you trying to view? --Mark [1] https://builds.apache.org/job/Geode-nightly/749/console [2] https://builds.apache.org/job/Geode-nightly/749/testReport/ [3] https://b

[GitHub] geode pull request #398: Split the redis adapter into its own package

Github user galen-pivotal commented on a diff in the pull request: https://github.com/apache/geode/pull/398#discussion_r101353985 --- Diff: geode-redis/src/test/java/org/apache/geode/redis/AuthJUnitTest.java --- @@ -0,0 +1,120 @@ +/* + * Licensed to the Apache Software Foun

[GitHub] geode pull request #398: Split the redis adapter into its own package

Github user galen-pivotal commented on a diff in the pull request: https://github.com/apache/geode/pull/398#discussion_r101354076 --- Diff: geode-redis/src/test/java/org/apache/geode/redis/AuthJUnitTest.java --- @@ -0,0 +1,120 @@ +/* + * Licensed to the Apache Software Foun

Re: Build failed in Jenkins: Geode-nightly #749

Hi Galen, I am able to the console log [1], test report summary [2], and artifacts [3] without logging in. Which 'build report' are you trying to view? --Mark [1] https://builds.apache.org/job/Geode-nightly/749/console [2] https://builds.apache.org/job/Geode-nightly/749/testReport/ [3] https://b

[jira] [Commented] (GEODE-2398) Sporadic Oplog corruption due to channel.write failure

[ https://issues.apache.org/jira/browse/GEODE-2398?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868350#comment-15868350 ] ASF subversion and git services commented on GEODE-2398: Commit fb

[jira] [Commented] (GEODE-2449) Move redis adapter to extension framework

[ https://issues.apache.org/jira/browse/GEODE-2449?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868352#comment-15868352 ] ASF subversion and git services commented on GEODE-2449: Commit a1

[jira] [Commented] (GEODE-2449) Move redis adapter to extension framework

[ https://issues.apache.org/jira/browse/GEODE-2449?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868353#comment-15868353 ] ASF subversion and git services commented on GEODE-2449: Commit 2f

[jira] [Updated] (GEODE-2488) Netstat command fails with OutOfMemoryError

[ https://issues.apache.org/jira/browse/GEODE-2488?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kirk Lund updated GEODE-2488: - Summary: Netstat command fails with OutOfMemoryError (was: NetstatDUnitTest fails with OutOfMemoryError)

[jira] [Assigned] (GEODE-2469) Redis adapter Hash key support

[ https://issues.apache.org/jira/browse/GEODE-2469?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Hitesh Khamesra reassigned GEODE-2469: -- Assignee: Hitesh Khamesra > Redis adapter Hash key support > --

[jira] [Updated] (GEODE-2488) Netstat command fails with OutOfMemoryError

[ https://issues.apache.org/jira/browse/GEODE-2488?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kirk Lund updated GEODE-2488: - Description: Note: this can occur outside of dunit tests as well. Just using the gfsh netstat command on

Re: Build failed in Jenkins: Geode-nightly #749

Hi Mark, I'm trying to view the Geode build report at this link: > There were failing tests. See the report at: file://< https://builds.apache.org/job/Geode-nightly/ws/geode-core/build/reports/ distributedTest/index.html> I don't see the geode-core/build/reports directory here: https://builds.apa

[jira] [Updated] (GEODE-2436) Geode doesn't handle byte[] as key

[ https://issues.apache.org/jira/browse/GEODE-2436?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Hitesh Khamesra updated GEODE-2436: --- Issue Type: Sub-task (was: Improvement) Parent: GEODE-2444 > Geode doesn't handle byt

[jira] [Updated] (GEODE-2469) Redis adapter Hash key support

[ https://issues.apache.org/jira/browse/GEODE-2469?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Hitesh Khamesra updated GEODE-2469: --- Issue Type: Sub-task (was: Bug) Parent: GEODE-2444 > Redis adapter Hash key support >

Re: Build failed in Jenkins: Geode-nightly #749

Galen - I'm guessing you might be trying to click on the link below? I'm not sure jenkins actually keeps that stuff - I think it's just looking for file:/// urls in the console output and translating them to http:// urls. You should be able to find the test results by going to the test result link

[jira] [Updated] (GEODE-2473) redis-cli hangs if Geode unable to process command properly

[ https://issues.apache.org/jira/browse/GEODE-2473?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Hitesh Khamesra updated GEODE-2473: --- Issue Type: Sub-task (was: Bug) Parent: GEODE-2444 > redis-cli hangs if Geode unable

[jira] [Commented] (GEODE-2449) Move redis adapter to extension framework

[ https://issues.apache.org/jira/browse/GEODE-2449?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868407#comment-15868407 ] ASF subversion and git services commented on GEODE-2449: Commit 2c

[jira] [Commented] (GEODE-2444) Redis Adapter Performance Improvements

[ https://issues.apache.org/jira/browse/GEODE-2444?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868408#comment-15868408 ] ASF subversion and git services commented on GEODE-2444: Commit 2c

[jira] [Resolved] (GEODE-2486) SSL ciphers other than NULL not supported

[ https://issues.apache.org/jira/browse/GEODE-2486?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jacob S. Barrett resolved GEODE-2486. - Resolution: Fixed Fix Version/s: 1.2.0 > SSL ciphers other than NULL not supported

[jira] [Resolved] (GEODE-2408) Refactor CacheableDate to use C++ std::chrono

[ https://issues.apache.org/jira/browse/GEODE-2408?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jacob S. Barrett resolved GEODE-2408. - Resolution: Fixed Fix Version/s: 1.2.0 > Refactor CacheableDate to use C++ std::chr

[jira] [Commented] (GEODE-2449) Move redis adapter to extension framework

[ https://issues.apache.org/jira/browse/GEODE-2449?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868419#comment-15868419 ] ASF subversion and git services commented on GEODE-2449: Commit 6d

[jira] [Created] (GEODE-2489) Tombstone message with keys are sent to peer partitioned region nodes even though no clinets are registered

Anilkumar Gingade created GEODE-2489: Summary: Tombstone message with keys are sent to peer partitioned region nodes even though no clinets are registered Key: GEODE-2489 URL: https://issues.apache.org/jira/br

[GitHub] geode issue #398: Split the redis adapter into its own package

Github user bschuchardt commented on the issue: https://github.com/apache/geode/pull/398 @metatype there are no changes in the netty NOTICE.txt between 4.1.7 and 4.1.8 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If y

Re: Build failed in Jenkins: Geode-nightly #749

ah, thanks Dan. I was looking in the wrong place. On Wed, Feb 15, 2017 at 11:12 AM, Dan Smith wrote: > Galen - I'm guessing you might be trying to click on the link below? I'm > not sure jenkins actually keeps that stuff - I think it's just looking for > file:/// urls in the console output and t

[GitHub] geode issue #398: Split the redis adapter into its own package

Github user galen-pivotal commented on the issue: https://github.com/apache/geode/pull/398 @metatype The import ordering is the rather unfortunate result of IntelliJ's import cleanup. --- If your project is set up for it, you can reply to this email and have your reply appear on Git

[jira] [Closed] (GEODE-84) Extract Redis adaptor from core

[ https://issues.apache.org/jira/browse/GEODE-84?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Swapnil Bawaskar closed GEODE-84. - > Extract Redis adaptor from core > --- > > Key: GEODE-84 >

[jira] [Resolved] (GEODE-84) Extract Redis adaptor from core

[ https://issues.apache.org/jira/browse/GEODE-84?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Swapnil Bawaskar resolved GEODE-84. --- Resolution: Duplicate > Extract Redis adaptor from core > --- > >

Re: GeodeRedisAdapter improvments/feedback

Hitesh and Team, Also, I think geospatial support in core Gemfire that can be exposed thru the following Redis GEO... commands would be great GEOADD GEODIST GEOHASH GEOPOS GEORADIUS GEORADIUSBYMEMBER On Wed, Feb 15, 2017 at 10:48 AM, Gregory Green wrote: > Hello Hitesh, > > The following is

[GitHub] geode issue #398: Split the redis adapter into its own package

Github user metatype commented on the issue: https://github.com/apache/geode/pull/398 Also, I believe redis is started by providing an option to `start server`. How do you plan to extract that from `geode-core`? --- If your project is set up for it, you can reply to this email and h

[GitHub] geode pull request #398: Split the redis adapter into its own package

Github user galen-pivotal commented on a diff in the pull request: https://github.com/apache/geode/pull/398#discussion_r101374277 --- Diff: geode-core/src/main/java/org/apache/geode/internal/cache/GemFireCacheImpl.java --- @@ -1330,21 +1360,12 @@ private void startMemcachedServer(

Re: PROXY and CACHING_PROXY regions on Client

@John The intention behind this proposal is to make Geode client development easy for new users. So, looking at this as a new user, I would say that having to "create" a PROXY region only to find out that it does nothing on the server, is more confusing than an overloaded getRegion(). To summarize

[jira] [Commented] (GEODE-2464) Review Redis tests

[ https://issues.apache.org/jira/browse/GEODE-2464?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868511#comment-15868511 ] Galen O'Sullivan commented on GEODE-2464: - We'll also need some manual tests to ma

[jira] [Assigned] (GEODE-2489) Tombstone message with keys are sent to peer partitioned region nodes even though no clinets are registered

[ https://issues.apache.org/jira/browse/GEODE-2489?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Anilkumar Gingade reassigned GEODE-2489: Assignee: Anilkumar Gingade > Tombstone message with keys are sent to peer partitio

[jira] [Created] (GEODE-2490) Tombstone messages are getting processed inline

Anilkumar Gingade created GEODE-2490: Summary: Tombstone messages are getting processed inline Key: GEODE-2490 URL: https://issues.apache.org/jira/browse/GEODE-2490 Project: Geode Issue T

[GitHub] geode pull request #398: Split the redis adapter into its own package

Github user metatype commented on a diff in the pull request: https://github.com/apache/geode/pull/398#discussion_r101375546 --- Diff: geode-core/src/main/java/org/apache/geode/internal/cache/GemFireCacheImpl.java --- @@ -1330,21 +1360,12 @@ private void startMemcachedServer() {

[jira] [Assigned] (GEODE-2490) Tombstone messages are getting processed inline

[ https://issues.apache.org/jira/browse/GEODE-2490?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Anilkumar Gingade reassigned GEODE-2490: Assignee: Anilkumar Gingade > Tombstone messages are getting processed inline > ---

Re: PROXY and CACHING_PROXY regions on Client

Re: breaking existing code; we could throw UnsupportedOperationException for these two methods: - invalidateRegion() - destroyRegion() I do not see anyone using/depending on invalidateRegion, since the behavior currently is a no-op. destroyRegion currently only gets rid of the handle to the region

Re: GeodeRedisAdapter improvments/feedback

>>>The Redis adapter was designed so that we can scale all the Redis data structures horizontally. If you bring the data structures to region entry level, there is no reason for anyone to use our implementation over Redis. hmm, here we need to understand when we need to create partition-region for

Re: GeodeRedisAdapter improvments/feedback

We should be careful here on a decision. If we start replicating fat lists/sets/hash maps synchronously every update, the Geode user will complain how slow the API is compared with Redis. Note: Redis replicates asynchronously. For fat collections we’re better off creating a region and suffer

[jira] [Assigned] (GEODE-2491) Reduce logging of handled exceptions in LuceneEventListener and LuceneBucketListeners

[ https://issues.apache.org/jira/browse/GEODE-2491?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jason Huynh reassigned GEODE-2491: -- Assignee: Jason Huynh > Reduce logging of handled exceptions in LuceneEventListener and > Luce

[jira] [Created] (GEODE-2491) Reduce logging of handled exceptions in LuceneEventListener and LuceneBucketListeners

Jason Huynh created GEODE-2491: -- Summary: Reduce logging of handled exceptions in LuceneEventListener and LuceneBucketListeners Key: GEODE-2491 URL: https://issues.apache.org/jira/browse/GEODE-2491 Proje

[jira] [Resolved] (GEODE-2424) afterSecondary call needs to handle specific exception rather than generic exception

[ https://issues.apache.org/jira/browse/GEODE-2424?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jason Huynh resolved GEODE-2424. Resolution: Duplicate Assignee: Jason Huynh Going to close this one as it is related to a larg

Re: Gradle build for idea

I think this sort of error message would indicate that we have something wrong in our gradle files. So, yes we should be concerned and fix it. Unfortunately, I don't know how to fix it... On Tue, Feb 14, 2017 at 1:36 PM, Udo Kohlmeyer wrote: > Ok... just to clarify... I have imported the projec

Review Request 56719: GEODE-2491: Reduce logging of handled exceptions in LuceneEventListener and LuceneBucketListeners

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56719/ --- Review request for geode, Lynn Hughes-Godfrey, nabarun nag, and Dan Smith. Repo

Re: GeodeRedisAdapter improvments/feedback

Thinking about this, I think that the “spill”/ “unspill” option may actually be the best solution. If the criteria waffles back and forth along the threshold, well, that’s the acceptable worst case. How’s this?: 1) Create a separate region for the collection key - for fat collections that

[GitHub] geode-native pull request #12: GEODE-2309: Remove or ignore apache-rat flagg...

GitHub user dgkimura opened a pull request: https://github.com/apache/geode-native/pull/12 GEODE-2309: Remove or ignore apache-rat flagged files `resolv.config` used to work around Solaris SPARC networking issues on a local OpenStack instance. It should not be needed going forward.

[jira] [Commented] (GEODE-2309) Replace or add ASF copyright statements in source.

[ https://issues.apache.org/jira/browse/GEODE-2309?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868561#comment-15868561 ] ASF GitHub Bot commented on GEODE-2309: --- GitHub user dgkimura opened a pull request:

Re: GeodeRedisAdapter improvments/feedback

With the suggestion from Wes, the constraint on the names would have to apply for both small and large. We wouldn't want the thing to explode when it gets converted... Is there a way to just make it configurable? If they know they want a "large" set, somehow let them specify it. Otherwise go wi

[GitHub] geode pull request #399: Minor non-functional changes in response to PR comm...

GitHub user galen-pivotal opened a pull request: https://github.com/apache/geode/pull/399 Minor non-functional changes in response to PR comments. @bschuchardt You can merge this pull request into a Git repository by running: $ git pull https://github.com/galen-pivotal/incubat

Re: GeodeRedisAdapter improvments/feedback

Doing the spill/unspill option could be pretty tricky to implement, so you have to do a lot of fancy logic in the transition period. I think Jason's suggestion of configuring things might make more sense. -Dan On Wed, Feb 15, 2017 at 1:12 PM, Jason Huynh wrote: > With the suggestion from Wes, t

Re: Review Request 56719: GEODE-2491: Reduce logging of handled exceptions in LuceneEventListener and LuceneBucketListeners

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56719/#review165761 --- geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal

Re: GeodeRedisAdapter improvments/feedback

Does delta propagation make worrying about frequently updated fat collections moot? On Feb 15, 2017, at 4:29 PM, Dan Smith mailto:dsm...@pivotal.io>> wrote: Doing the spill/unspill option could be pretty tricky to implement, so you have to do a lot of fancy logic in the transition period. I th

[GitHub] geode pull request #398: Split the redis adapter into its own package

Github user galen-pivotal commented on a diff in the pull request: https://github.com/apache/geode/pull/398#discussion_r101388584 --- Diff: geode-core/src/main/java/org/apache/geode/internal/cache/GemFireCacheImpl.java --- @@ -1330,21 +1360,12 @@ private void startMemcachedServer(

[GitHub] geode pull request #398: Split the redis adapter into its own package

Github user galen-pivotal commented on a diff in the pull request: https://github.com/apache/geode/pull/398#discussion_r101388974 --- Diff: geode-core/src/main/java/org/apache/geode/internal/cache/GemFireCacheImpl.java --- @@ -1330,21 +1360,12 @@ private void startMemcachedServer(

Re: Review Request 56719: GEODE-2491: Reduce logging of handled exceptions in LuceneEventListener and LuceneBucketListeners

> On Feb. 15, 2017, 9:33 p.m., nabarun nag wrote: > > geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/LuceneBucketListener.java, > > line 58 > > > > > > Hi Jason, > > > > I am not sure but

[GitHub] geode-native pull request #13: GEODE-2476: Replace gfcpp with geode.

GitHub user PivotalSarge opened a pull request: https://github.com/apache/geode-native/pull/13 GEODE-2476: Replace gfcpp with geode. - Rename directories and files with gfcpp into their name to instead use geode and update all references thereto. - Rename the gfcpp executab

[jira] [Commented] (GEODE-2476) Replace gfcpp with geode

[ https://issues.apache.org/jira/browse/GEODE-2476?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15868619#comment-15868619 ] ASF GitHub Bot commented on GEODE-2476: --- GitHub user PivotalSarge opened a pull requ

Re: [DISCUSS] JIRA guidelines

The draft bit has been removed. Thanks to William for writing these in the first place! -Dan On Wed, Feb 15, 2017 at 8:16 AM, Michael William Dodge wrote: > +1 > > > On 14 Feb, 2017, at 20:50, William Markito Oliveira < > william.mark...@gmail.com> wrote: > > > > +1 > > > > Finally!! ;) > > > >

Re: Review Request 56719: GEODE-2491: Reduce logging of handled exceptions in LuceneEventListener and LuceneBucketListeners

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56719/#review165766 --- Ship it! Ship It! - nabarun nag On Feb. 15, 2017, 8:54 p.m.,

[GitHub] geode pull request #398: Split the redis adapter into its own package

Github user metatype commented on a diff in the pull request: https://github.com/apache/geode/pull/398#discussion_r101391406 --- Diff: geode-core/src/main/java/org/apache/geode/internal/cache/GemFireCacheImpl.java --- @@ -1330,21 +1360,12 @@ private void startMemcachedServer() {

  1   2   >