Re: Simple Java Client

I like the idea of a separate code base (including repo) for separate problem domains. I think trying to fit a new API inside or parallel to the existing API has too high of a likelihood of confusion. Sarge > On 26 Apr, 2017, at 16:52, Bruce Schuchardt wrote: > > I

[GitHub] geode-examples issue #4: Updating and simplifying examples

Github user metatype commented on the issue: https://github.com/apache/geode-examples/pull/4 The latest update incorporates the feedback on testing. --- 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

Re: Simple Java Client

I don't think we should mix the old client code with a new API. We should keep the new client code separate from the server. Maybe even in a different repo, as I think Fred suggested. Le 4/26/2017 à 3:12 PM, Kirk Lund a écrit : If we want to add a new API then I suggest we create new API

[jira] [Created] (GEODE-2831) Make it easier to create an example

Anthony Baker created GEODE-2831: Summary: Make it easier to create an example Key: GEODE-2831 URL: https://issues.apache.org/jira/browse/GEODE-2831 Project: Geode Issue Type: Improvement

[jira] [Resolved] (GEODE-2801) disk store backup can fail with ArrayIndexOutOfBoundsException

[ https://issues.apache.org/jira/browse/GEODE-2801?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Darrel Schneider resolved GEODE-2801. - Resolution: Fixed Fix Version/s: 1.2.0 > disk store backup can fail with

[jira] [Commented] (GEODE-2801) disk store backup can fail with ArrayIndexOutOfBoundsException

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

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

--- Spring Data GemFire > Nightly-ApacheGeode > #536 was successful. --- Scheduled 1845 tests in total. https://build.spring.io/browse/SGF-NAG-536/ -- This

Re: Simple Java Client

If we want to add a new API then I suggest we create new API packages and leave the "cache" package as the old deprecated API for backwards compatibility. The new APIs and implementation could be separated into different geode modules and packages (something like what follows):

[jira] [Commented] (GEODE-2826) Client docs - Delta Propagation API, correct .NET namespace notation

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

[jira] [Commented] (GEODE-2826) Client docs - Delta Propagation API, correct .NET namespace notation

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

[GitHub] geode-native pull request #94: GEODE-2826: Client docs - Delta Propagation A...

Github user asfgit closed the pull request at: https://github.com/apache/geode-native/pull/94 --- 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] [Commented] (GEODE-2828) AEQ needs to be created before the user region

[ https://issues.apache.org/jira/browse/GEODE-2828?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15985589#comment-15985589 ] ASF GitHub Bot commented on GEODE-2828: --- Github user jhuynh1 commented on a diff in the pull

[GitHub] geode pull request #479: GEODE-2828: AEQ created before the user region

Github user jhuynh1 commented on a diff in the pull request: https://github.com/apache/geode/pull/479#discussion_r113565230 --- Diff: geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/LuceneServiceImpl.java --- @@ -166,28 +166,28 @@ public void createIndex(final

[jira] [Commented] (GEODE-2828) AEQ needs to be created before the user region

[ https://issues.apache.org/jira/browse/GEODE-2828?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15985582#comment-15985582 ] ASF GitHub Bot commented on GEODE-2828: --- Github user upthewaterspout commented on a diff in the pull

[GitHub] geode pull request #479: GEODE-2828: AEQ created before the user region

Github user upthewaterspout commented on a diff in the pull request: https://github.com/apache/geode/pull/479#discussion_r113564312 --- Diff: geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/LuceneEventListener.java --- @@ -53,6 +54,8 @@ Logger logger =

Re: Review Request 58682: GEODE-2662: Missing keys cause columns to shift in gfsh table display.

> On April 25, 2017, 9:10 p.m., Kirk Lund wrote: > > geode-core/src/test/java/org/apache/geode/management/internal/cli/functions/DataCommandFunctionWithPDXJUnitTest.java > > Lines 154 (patched) > > > > > > Usually

Re: Review Request 58682: GEODE-2662: Missing keys cause columns to shift in gfsh table display.

> On April 25, 2017, 9:10 p.m., Kirk Lund wrote: > > geode-core/src/main/java/org/apache/geode/management/internal/cli/functions/DataCommandFunction.java > > Line 301 (original), 317 (patched) > > > > > > Should

Re: Review Request 58589: GEODE-1597: use Spring shell's parser and delete our own parsing code

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58589/#review173100 --- Ship it! Ship It! - Kirk Lund On April 26, 2017, 7:48 p.m.,

[GitHub] geode pull request #476: GEODE-2816: Redundancy recovery inititiated even if...

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

Re: Review Request 58589: GEODE-1597: use Spring shell's parser and delete our own parsing code

> On April 25, 2017, 5:43 p.m., Kirk Lund wrote: > > geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/ShellCommands.java > > Lines 238 (patched) > > > > > > Insert one line above the throw

[GitHub] geode issue #479: GEM-1331: AEQ created before the user region

Github user nabarunnag commented on the issue: https://github.com/apache/geode/pull/479 Pushed to develop Failure in Travis fixed in GEODE-2632 --- 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

Re: ASF review board down?

It's working for me now too. On Wed, Apr 26, 2017 at 1:56 PM, Dan Smith wrote: > Works for me now. > > -Dan > > On Wed, Apr 26, 2017 at 11:22 AM, Kirk Lund wrote: > > > I'm unable to submit any review board reviews on > > https://reviews.apache.org/. > > I

[GitHub] geode pull request #479: GEM-1331: AEQ created before the user region

GitHub user nabarunnag reopened a pull request: https://github.com/apache/geode/pull/479 GEM-1331: AEQ created before the user region Potential reviewers @gesterzhou @upthewaterspout @jhuynh1 @ladyVader You can merge this pull request into a Git repository by running: $

[GitHub] geode pull request #479: GEM-1331: AEQ created before the user region

Github user nabarunnag closed the pull request at: https://github.com/apache/geode/pull/479 --- 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-2816) Redundancy recovery must also kick in when redundancy recovery is set to 0

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

Re: Review Request 58752: GEODE-236: Removed deprecated CacheEvent methods

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58752/#review173096 --- Ship it! Ship It! - Darrel Schneider On April 26, 2017,

Re: Review Request 58742: GEODE-2632: minor fixes from code review

> On April 26, 2017, 5:51 p.m., Jinmei Liao wrote: > > geode-core/src/main/java/org/apache/geode/internal/cache/GemFireCacheImpl.java > > Line 2411 (original), 2412 (patched) > > > > > > I know this is the original

Re: ASF review board down?

Works for me now. -Dan On Wed, Apr 26, 2017 at 11:22 AM, Kirk Lund wrote: > I'm unable to submit any review board reviews on > https://reviews.apache.org/. > I was able to submit a review an hour or so ago, but now I get this cryptic > message: > > "fatal: git cat-file: could

[jira] [Commented] (GEODE-2821) Geode User Guide: Add running heads with logo

[ https://issues.apache.org/jira/browse/GEODE-2821?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15985537#comment-15985537 ] ASF GitHub Bot commented on GEODE-2821: --- Github user davebarnes97 commented on the issue:

Review Request 58752: GEODE-236: Removed deprecated CacheEvent methods

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58752/ --- Review request for geode. Repository: geode Description ---

[jira] [Commented] (GEODE-2821) Geode User Guide: Add running heads with logo

[ https://issues.apache.org/jira/browse/GEODE-2821?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15985538#comment-15985538 ] ASF GitHub Bot commented on GEODE-2821: --- Github user davebarnes97 closed the pull request at:

Re: Finer grained security

I agree that async event queues seem like a different case than wan or disk. In that case you are not using anything that creating a region doesn't do. Shouldn't creating a region be DATA:MANAGE:DISK? Requiring DATA privileges for a region without disk and CLUSTER privileges for a region with

Review Request 58751: GEODE-2632: make GemFireCacheImpl.getRegion(String) non-final

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58751/ --- Review request for geode, Barry Oglesby, Jinmei Liao, Jared Stewart, Ken Howe,

[jira] [Commented] (GEODE-2821) Geode User Guide: Add running heads with logo

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

[jira] [Commented] (GEODE-2825) Lucene query may stackoverflow if enough function execution retries are executed

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

[GitHub] geode pull request #480: GEODE-2825: Lucene query waits for defined index to...

GitHub user jhuynh1 opened a pull request: https://github.com/apache/geode/pull/480 GEODE-2825: Lucene query waits for defined index to be created * If an index is in a defined state but not yet created, the query will now wait until the index is created or no longer

Re: Review Request 58742: GEODE-2632: minor fixes from code review

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58742/#review173094 --- Ship it! Ship It! - Patrick Rhomberg On April 26, 2017,

Re: Finer grained security

One more possible complication is that creating a Lucene index will also create an AsyncEventQueue. Today the required permission to create the AEQ is DATA:MANAGE which coincidentally nicely matches the permission required to create an OQL index. Pulling out the AEQ as a separate resource will

[jira] [Created] (GEODE-2830) Required permission for executing a function should be DATA:WRITE

Diane Hardman created GEODE-2830: Summary: Required permission for executing a function should be DATA:WRITE Key: GEODE-2830 URL: https://issues.apache.org/jira/browse/GEODE-2830 Project: Geode

Re: Review Request 58589: GEODE-1597: use Spring shell's parser and delete our own parsing code

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58589/ --- (Updated April 26, 2017, 7:48 p.m.) Review request for geode, Jared Stewart,

[jira] [Created] (GEODE-2829) VMRegionVersionVector allows/stores DiskStoreId as its member id

Anilkumar Gingade created GEODE-2829: Summary: VMRegionVersionVector allows/stores DiskStoreId as its member id Key: GEODE-2829 URL: https://issues.apache.org/jira/browse/GEODE-2829 Project:

[jira] [Commented] (GEODE-2802) TombstoneMessage can throw SerializationException when region is configured as persistent and non-persistent in cluster (in different nodes).

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

ParallelQueueRemovalMessageJUnitTest

FYI, I broke ParallelQueueRemovalMessageJUnitTest with a recent checkin and then I committed a temporary fix. So it should be back to passing. I now have a more complete fix ready to submit for review, but I'm unable to submit new requests on https://reviews.apache.org. Not sure what's wrong

Re: Review Request 58682: GEODE-2662: Missing keys cause columns to shift in gfsh table display.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58682/ --- (Updated April 26, 2017, 6:25 p.m.) Review request for geode, Jinmei Liao,

ASF review board down?

I'm unable to submit any review board reviews on https://reviews.apache.org/. I was able to submit a review an hour or so ago, but now I get this cryptic message: "fatal: git cat-file: could not get object info Line undefined: undefined" Is anyone else still able to submit a review? The diff

Re: Review Request 58682: GEODE-2662: Missing keys cause columns to shift in gfsh table display.

> On April 25, 2017, 9:10 p.m., Kirk Lund wrote: > > geode-core/src/main/java/org/apache/geode/management/internal/cli/functions/DataCommandFunction.java > > Line 301 (original), 317 (patched) > > > > > > Should

[jira] [Commented] (GEODE-2825) Lucene query may stackoverflow if enough function execution retries are executed

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

[jira] [Commented] (GEODE-2632) Integrated Security performance improvements

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

Re: Review Request 58742: GEODE-2632: minor fixes from code review

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58742/#review173066 --- Ship it! Ship It! - Jinmei Liao On April 26, 2017, 5:17

Re: Simple Java Client

The conservation on this thread is getting more focused on adopting JCache (JSR107) standard... Fred started this conversation to see if some of the admin APIs can be supported as part of Java API, to help devs who wants to combine both admin and business logic in the same application...

Review Request 58742: GEODE-2632: minor fixes from code review

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58742/ --- Review request for geode, Jinmei Liao, Jared Stewart, Ken Howe, and Patrick

[GitHub] geode pull request #479: GEM-1331: AEQ created before the user region

GitHub user nabarunnag opened a pull request: https://github.com/apache/geode/pull/479 GEM-1331: AEQ created before the user region Potential reviewers @gesterzhou @upthewaterspout @jhuynh1 @ladyVader You can merge this pull request into a Git repository by running: $ git

[jira] [Commented] (GEODE-2632) Integrated Security performance improvements

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

[jira] [Assigned] (GEODE-2661) CacheListener gets invoked when an non-existent entry is removed using removeAll

[ https://issues.apache.org/jira/browse/GEODE-2661?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Lynn Gallinat reassigned GEODE-2661: Assignee: Lynn Gallinat > CacheListener gets invoked when an non-existent entry is removed

Re: Simple Java Client

This is an awesome discussion and Jake's hitting all the right notes about why JCache is a good idea! I've fought that fight in the past and lost it but I'm happy to see it coming back... What's really nice about Geode is that the functionalities and capabilities are all there, they're just not

Re: Review Request 58712: GEODE-2632: partial cleanup of GemFireCacheImpl

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58712/ --- (Updated April 26, 2017, 4:39 p.m.) Review request for geode, Darrel

[jira] [Resolved] (GEODE-2689) If a region containing a Lucene index is created in one group and altered in another, a member in the other group will fail to start

[ https://issues.apache.org/jira/browse/GEODE-2689?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Barry Oglesby resolved GEODE-2689. -- Resolution: Fixed Fix Version/s: 1.2.0 > If a region containing a Lucene index is

[jira] [Commented] (GEODE-2689) If a region containing a Lucene index is created in one group and altered in another, a member in the other group will fail to start

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

[jira] [Resolved] (GEODE-2813) Incorrect Error Message for REST API, Doc Doesn

[ https://issues.apache.org/jira/browse/GEODE-2813?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dave Barnes resolved GEODE-2813. Resolution: Duplicate Duplicate of GEODE-2815. > Incorrect Error Message for REST API, Doc Doesn >

[jira] [Closed] (GEODE-2813) Incorrect Error Message for REST API, Doc Doesn

[ https://issues.apache.org/jira/browse/GEODE-2813?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dave Barnes closed GEODE-2813. -- > Incorrect Error Message for REST API, Doc Doesn > --- > >

Re: Simple Java Client

Wes, Those are almost all administrative commands and have no place on the client API. They belong on an administrative API or as I'm arguing a series of MBeans/JMX as it is already an established standard. -Jake On Wed, Apr 26, 2017 at 8:09 AM Wes Williams wrote: > Now

Re: Simple Java Client

Now we're getting some precision. Let's talk about the "raw" Geode proprietary bad ass API! Would that "raw" Geode proprietary bad ass API "raw" Geode proprietary bad ass API that we're talking about be centered around the commands found here:

[jira] [Commented] (GEODE-2681) Gfsh show missing-disk-stores command hangs when server is waiting for disk stores

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

[jira] [Commented] (GEODE-253) Remove deprecated EntryNotFoundInRegion

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

[jira] [Commented] (GEODE-253) Remove deprecated EntryNotFoundInRegion

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

[GitHub] geode pull request #465: GEODE-253: Removed depreated and not used EntryNotF...

Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/465 --- 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-255) Remove deprecated DataSerializer.register(Class,byte)

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

[jira] [Commented] (GEODE-255) Remove deprecated DataSerializer.register(Class,byte)

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

[jira] [Commented] (GEODE-260) Remove deprecated RemoteTransactionException

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

[jira] [Commented] (GEODE-260) Remove deprecated RemoteTransactionException

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

[GitHub] geode pull request #468: GEODE-260: Remove deprecated RemoteTransactionExcep...

Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/468 --- 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-266) Remove deprecated ObjectSizerImpl

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

[GitHub] geode pull request #469: GEODE-266: Remove the deprecated ObjectSizerImpl cl...

Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/469 --- 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-266) Remove deprecated ObjectSizerImpl

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