[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] [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=15868353#comment-15868353 ] ASF subversion and git services commented on GEODE-2449: Commit

[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=15868352#comment-15868352 ] ASF subversion and git services commented on GEODE-2449: Commit

[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

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]

[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

[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=15868351#comment-15868351 ] ASF subversion and git services commented on GEODE-2449: Commit

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]

[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=15868350#comment-15868350 ] ASF subversion and git services commented on GEODE-2398: Commit

[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

[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=15868347#comment-15868347 ] ASF subversion and git services commented on GEODE-2475: Commit

[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=15868348#comment-15868348 ] ASF subversion and git services commented on GEODE-2398: Commit

[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=15868344#comment-15868344 ] Eric Shu commented on GEODE-2485: - Stack trace for the above scenario: {noformat} at

[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=15868337#comment-15868337 ] ASF subversion and git services commented on GEODE-2484: Commit

[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

[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

[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=15868328#comment-15868328 ] ASF GitHub Bot commented on GEODE-2486: --- Github user asfgit closed the pull request at:

[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=15868277#comment-15868277 ] Michael Martell edited comment on GEODE-2421 at 2/15/17 6:45 PM: - Suggest

[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

[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

[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=15868317#comment-15868317 ] Kirk Lund edited comment on GEODE-2488 at 2/15/17 6:37 PM: --- NetstatFunction

[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=15868317#comment-15868317 ] Kirk Lund commented on GEODE-2488: -- NetstatFunction reads in every line from the netstat output and lsof

[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=15868312#comment-15868312 ] Eric Shu commented on GEODE-2485: - Suspend and resume could be called from product. For each create on a

[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

[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] [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] [Commented] (GEODE-2421) Create VS2015 AMI

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

[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

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,

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

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

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

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

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

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

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

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

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

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

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

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

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

[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=15867917#comment-15867917 ] ASF GitHub Bot commented on GEODE-2486: --- Github user echobravopapa commented on a diff in the pull

[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

<    1   2