Re: [gemfire-dev] New Client-Server Protocol Proposal

2017-05-03 Thread Dan Smith
+1 to what Anthony has laid out! I think this is a better way to handle value encodings, and it's also better to be putting message specific details like event id with those messages. I do wonder whether this proposal actually needs metadata headers at all? What will eventually go in there? -Dan

[jira] [Commented] (GEODE-254) Remove deprecated Region.keys and Region.entries

2017-05-03 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15996004#comment-15996004 ] ASF GitHub Bot commented on GEODE-254: -- Github user asfgit closed the pull request at:

[jira] [Commented] (GEODE-254) Remove deprecated Region.keys and Region.entries

2017-05-03 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15996002#comment-15996002 ] ASF subversion and git services commented on GEODE-254: --- Commit

[GitHub] geode pull request #488: GEODE-254: Removed deprecated Region.keys and Regio...

2017-05-03 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/488 --- 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: [gemfire-dev] New Client-Server Protocol Proposal

2017-05-03 Thread Hitesh Khamesra
Good point Dan !! that needs to document. From: Dan Smith To: dev@geode.apache.org Sent: Wednesday, May 3, 2017 5:31 PM Subject: Re: [gemfire-dev] New Client-Server Protocol Proposal Okay but how do I has an implementer of a driver know what messages need

[jira] [Commented] (GEODE-2754) CI failure: WanAutoDiscoveryDUnitTest

2017-05-03 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2754?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15995982#comment-15995982 ] ASF GitHub Bot commented on GEODE-2754: --- GitHub user nabarunnag opened a pull request:

[GitHub] geode pull request #492: GEODE-2754: Changed the name of unknown host

2017-05-03 Thread nabarunnag
GitHub user nabarunnag opened a pull request: https://github.com/apache/geode/pull/492 GEODE-2754: Changed the name of unknown host * Assert failures occured if the network contained machines named "unknown" as the test function also named the unknown host as "unknown"

Re: [gemfire-dev] New Client-Server Protocol Proposal

2017-05-03 Thread Anthony Baker
> On May 3, 2017, at 1:33 PM, Galen M O'Sullivan wrote: > > On Tue, May 2, 2017 at 11:52 AM, Hitesh Khamesra < > hitesh...@yahoo.com.invalid> wrote: > >> Absolutely its a implementation detail. >> > This doesn't answer Dan's comment. Do you think fragmentation should be

Re: [gemfire-dev] New Client-Server Protocol Proposal

2017-05-03 Thread Dan Smith
Okay but how do I has an implementer of a driver know what messages need an event id and which don't? It seems like maybe this belongs with those message types, rather than in a generic header. Or maybe you need to start organizing messages into classes - eg messages that change state and

[jira] [Updated] (GEODE-2848) While destroying a LuceneIndex, the AsyncEventQueue region is destroyed in remote members before stopping the AsyncEventQueue

2017-05-03 Thread xiaojian zhou (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2848?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] xiaojian zhou updated GEODE-2848: - Fix Version/s: 1.2.0 > While destroying a LuceneIndex, the AsyncEventQueue region is destroyed in

Re: [gemfire-dev] New Client-Server Protocol Proposal

2017-05-03 Thread Udo Kohlmeyer
Correct, I did miss that. @Dan, if you look at https://cwiki.apache.org/confluence/display/GEODE/Message+Structure+and+Definition#MessageStructureandDefinition-MetaDataforRequests specifies how we provide EventId information. On 5/3/17 09:53, Bruce Schuchardt wrote: I believe Hitesh put

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

2017-05-03 Thread Jason Huynh (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2825?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jason Huynh resolved GEODE-2825. Resolution: Fixed Fix Version/s: 1.2.0 > Lucene query may stackoverflow if enough function

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

2017-05-03 Thread Jason Huynh (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2825?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jason Huynh reassigned GEODE-2825: -- Assignee: Jason Huynh > Lucene query may stackoverflow if enough function execution retries

[jira] [Resolved] (GEODE-2837) Start server fails if specified `--dir` does not exist

2017-05-03 Thread Jared Stewart (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2837?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jared Stewart resolved GEODE-2837. -- Resolution: Fixed > Start server fails if specified `--dir` does not exist >

[jira] [Updated] (GEODE-2837) Start server fails if specified `--dir` does not exist

2017-05-03 Thread Jared Stewart (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2837?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jared Stewart updated GEODE-2837: - Fix Version/s: 1.2.0 > Start server fails if specified `--dir` does not exist >

Re: can cache-1.0.xsd be modified?

2017-05-03 Thread Jacob Barrett
The version number should match the version of the product at least up into the minor point. If you're adding new features you should be rolling in minor point of the product version number anyway. So if the feature is a 1.1 feature you don't want to put it on the 1.0 schema because of somebody

[jira] [Commented] (GEODE-2870) BucketMovedException during function execution may lead to client missing results

2017-05-03 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2870?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15995893#comment-15995893 ] ASF subversion and git services commented on GEODE-2870: Commit

[GitHub] geode pull request #475: GEODE-2812: Add API to get list of live locators

2017-05-03 Thread kirklund
Github user kirklund commented on a diff in the pull request: https://github.com/apache/geode/pull/475#discussion_r114674811 --- Diff: geode-core/src/main/java/org/apache/geode/cache/client/Pool.java --- @@ -193,13 +193,23 @@ /** * Returns an unmodifiable list of

[jira] [Commented] (GEODE-2812) Add API to get list of live locators

2017-05-03 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2812?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15995887#comment-15995887 ] ASF GitHub Bot commented on GEODE-2812: --- Github user kirklund commented on a diff in the pull

[jira] [Commented] (GEODE-2812) Add API to get list of live locators

2017-05-03 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2812?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15995888#comment-15995888 ] ASF GitHub Bot commented on GEODE-2812: --- Github user kirklund commented on a diff in the pull

[jira] [Commented] (GEODE-2870) BucketMovedException during function execution may lead to client missing results

2017-05-03 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2870?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15995885#comment-15995885 ] ASF GitHub Bot commented on GEODE-2870: --- Github user jhuynh1 commented on a diff in the pull

[GitHub] geode pull request #491: GEODE-2870: Local node function execution failure c...

2017-05-03 Thread jhuynh1
Github user jhuynh1 commented on a diff in the pull request: https://github.com/apache/geode/pull/491#discussion_r114674904 --- Diff: geode-core/src/main/java/org/apache/geode/internal/cache/execute/PartitionedRegionFunctionResultSender.java --- @@ -214,16 +214,15 @@ public void

[jira] [Commented] (GEODE-2853) Change of locator list request interval

2017-05-03 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2853?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15995876#comment-15995876 ] ASF GitHub Bot commented on GEODE-2853: --- Github user kirklund commented on a diff in the pull

[GitHub] geode pull request #483: GEODE-2853: Change of locator list request interval

2017-05-03 Thread kirklund
Github user kirklund commented on a diff in the pull request: https://github.com/apache/geode/pull/483#discussion_r114674582 --- Diff: geode-core/src/test/java/org/apache/geode/cache/client/internal/LocatorTestBase.java --- @@ -289,6 +289,13 @@ public void run() throws Exception

[GitHub] geode pull request #490: Feature/geode 2852

2017-05-03 Thread kirklund
Github user kirklund commented on a diff in the pull request: https://github.com/apache/geode/pull/490#discussion_r114673640 --- Diff: geode-core/src/main/java/org/apache/geode/internal/cache/wan/parallel/WaitUntilParallelGatewaySenderFlushedCoordinator.java --- @@ -46,22 +48,30

[jira] [Updated] (GEODE-2790) DiskSpaceLimitIntegrationTest.aboveZeroDeletesPreviousFiles_nameWithHyphen fails intermittently with ComparisonFailure

2017-05-03 Thread Kirk Lund (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2790?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kirk Lund updated GEODE-2790: - Labels: Flaky (was: ) > DiskSpaceLimitIntegrationTest.aboveZeroDeletesPreviousFiles_nameWithHyphen >

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-03 Thread Ken Howe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5/#review173845 ---

[jira] [Commented] (GEODE-2870) BucketMovedException during function execution may lead to client missing results

2017-05-03 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2870?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15995861#comment-15995861 ] ASF GitHub Bot commented on GEODE-2870: --- Github user kirklund commented on a diff in the pull

[GitHub] geode pull request #491: GEODE-2870: Local node function execution failure c...

2017-05-03 Thread kirklund
Github user kirklund commented on a diff in the pull request: https://github.com/apache/geode/pull/491#discussion_r114672970 --- Diff: geode-core/src/main/java/org/apache/geode/internal/cache/execute/PartitionedRegionFunctionResultSender.java --- @@ -214,16 +214,15 @@ public void

[jira] [Resolved] (GEODE-2857) Add getProcess to ProcessWrapper

2017-05-03 Thread Kirk Lund (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2857?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kirk Lund resolved GEODE-2857. -- Resolution: Fixed > Add getProcess to ProcessWrapper > > >

[jira] [Updated] (GEODE-2857) Add getProcess to ProcessWrapper

2017-05-03 Thread Kirk Lund (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2857?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kirk Lund updated GEODE-2857: - Fix Version/s: 1.2.0 > Add getProcess to ProcessWrapper > > >

[jira] [Resolved] (GEODE-2875) shutdown is taking as long as 20 seconds

2017-05-03 Thread Bruce Schuchardt (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2875?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Bruce Schuchardt resolved GEODE-2875. - Resolution: Fixed > shutdown is taking as long as 20 seconds >

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

2017-05-03 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #543 was successful. --- Scheduled 1845 tests in total. https://build.spring.io/browse/SGF-NAG-543/ -- This

[jira] [Commented] (GEODE-2837) Start server fails if specified `--dir` does not exist

2017-05-03 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15995818#comment-15995818 ] ASF subversion and git services commented on GEODE-2837: Commit

[jira] [Commented] (GEODE-2837) Start server fails if specified `--dir` does not exist

2017-05-03 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15995817#comment-15995817 ] ASF subversion and git services commented on GEODE-2837: Commit

[jira] [Resolved] (GEODE-2747) Investigate Binary Encoding Options

2017-05-03 Thread Addison (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2747?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Addison resolved GEODE-2747. Resolution: Fixed See protocol proposal > Investigate Binary Encoding Options >

[jira] [Resolved] (GEODE-2864) In same cases committing a geode transaction logs a meaningless warning

2017-05-03 Thread Darrel Schneider (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2864?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Darrel Schneider resolved GEODE-2864. - Resolution: Fixed Fix Version/s: 1.2.0 > In same cases committing a geode

[jira] [Commented] (GEODE-2864) In same cases committing a geode transaction logs a meaningless warning

2017-05-03 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2864?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15995807#comment-15995807 ] ASF subversion and git services commented on GEODE-2864: Commit

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-03 Thread Ken Howe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5/#review173839 ---

Re: Review Request 58813: GEODE-2776 After region is updated with loader, the ClientEvent is set with current entry version tag

2017-05-03 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58813/#review173838 --- Ship it! Ship It! - Darrel Schneider On May 3, 2017, 3:14

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-03 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5/#review173837 ---

[jira] [Commented] (GEODE-254) Remove deprecated Region.keys and Region.entries

2017-05-03 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15995795#comment-15995795 ] ASF GitHub Bot commented on GEODE-254: -- Github user dschneider-pivotal commented on the issue:

[GitHub] geode issue #488: GEODE-254: Removed deprecated Region.keys and Region.entri...

2017-05-03 Thread dschneider-pivotal
Github user dschneider-pivotal commented on the issue: https://github.com/apache/geode/pull/488 This looks good. Pull it in. --- 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

Re: Review Request 58813: GEODE-2776 After region is updated with loader, the ClientEvent is set with current entry version tag

2017-05-03 Thread anilkumar gingade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58813/ --- (Updated May 3, 2017, 10:14 p.m.) Review request for geode, Darrel Schneider,

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-03 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5/ --- (Updated May 3, 2017, 10:10 p.m.) Review request for geode, Darrel Schneider,

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-03 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5/ --- (Updated May 3, 2017, 10:02 p.m.) Review request for geode, Darrel Schneider,

[jira] [Commented] (GEODE-2875) shutdown is taking as long as 20 seconds

2017-05-03 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2875?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15995747#comment-15995747 ] ASF subversion and git services commented on GEODE-2875: Commit

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-03 Thread Ken Howe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5/#review173821 ---

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-03 Thread Jared Stewart
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5/#review173832 ---

[jira] [Created] (GEODE-2875) shutdown is taking as long as 20 seconds

2017-05-03 Thread Bruce Schuchardt (JIRA)
Bruce Schuchardt created GEODE-2875: --- Summary: shutdown is taking as long as 20 seconds Key: GEODE-2875 URL: https://issues.apache.org/jira/browse/GEODE-2875 Project: Geode Issue Type: Bug

[jira] [Commented] (GEODE-2852) Enforce WaitUntilFlushedFunction.waitUntilFlushed() timeout across all (local) buckets (not per bucket)

2017-05-03 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2852?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15995693#comment-15995693 ] ASF GitHub Bot commented on GEODE-2852: --- GitHub user ladyVader opened a pull request:

[jira] [Commented] (GEODE-2870) BucketMovedException during function execution may lead to client missing results

2017-05-03 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2870?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15995694#comment-15995694 ] ASF GitHub Bot commented on GEODE-2870: --- GitHub user jhuynh1 opened a pull request:

[GitHub] geode pull request #491: GEODE-2870: Local node function execution failure c...

2017-05-03 Thread jhuynh1
GitHub user jhuynh1 opened a pull request: https://github.com/apache/geode/pull/491 GEODE-2870: Local node function execution failure correctly returns e… …xception * Race condition and escaping synchronized block led to function possibly missing results * It was

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-03 Thread Kirk Lund
> On May 2, 2017, 9:57 p.m., Jared Stewart wrote: > > geode-core/src/main/java/org/apache/geode/distributed/internal/InternalLocator.java > > Line 1230 (original), 1187 (patched) > > > > > > This looks like a

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-03 Thread Kirk Lund
> On May 2, 2017, 8:36 p.m., Jared Stewart wrote: > > geode-core/src/main/java/org/apache/geode/cache/query/internal/index/HashIndex.java > > Line 549 (original), 483 (patched) > > > > > > These two booleans (`asc`

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-03 Thread Kirk Lund
> On May 3, 2017, 6:35 p.m., Ken Howe wrote: > > geode-core/src/main/java/org/apache/geode/cache/query/internal/index/CompactRangeIndex.java > > Lines 910-913 (original), 847-850 (patched) > > > > > > If I

[jira] [Resolved] (GEODE-2848) While destroying a LuceneIndex, the AsyncEventQueue region is destroyed in remote members before stopping the AsyncEventQueue

2017-05-03 Thread xiaojian zhou (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2848?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] xiaojian zhou resolved GEODE-2848. -- Resolution: Fixed fix in revision d4ece31fa23bbe74c8be0a82ff4b9d143bad79b3 > While destroying

[GitHub] geode pull request #487: geode-2848: when found all the partitioned region i...

2017-05-03 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/487 --- 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 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-03 Thread Kirk Lund
> On May 2, 2017, 5:24 p.m., Jared Stewart wrote: > > extensions/geode-modules/src/main/java/org/apache/geode/modules/session/catalina/DeltaSessionManager.java > > Line 301 (original), 310 (patched) > > > > > > Is

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-03 Thread Kirk Lund
> On May 2, 2017, 10:46 p.m., Jared Stewart wrote: > > geode-core/src/main/java/org/apache/geode/internal/cache/ColocationHelper.java > > Line 137 (original), 139 (patched) > > > > > > Why was `"/"` changed to

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-03 Thread Kirk Lund
> On May 2, 2017, 5:50 p.m., Ken Howe wrote: > > geode-core/src/main/java/org/apache/geode/cache/query/internal/CompiledLike.java > > Line 39 (original), 37 (patched) > > > > > > Does logger need to be static? By

[jira] [Created] (GEODE-2874) StringIndexOutOfBoundsException while initializing logger

2017-05-03 Thread Hitesh Khamesra (JIRA)
Hitesh Khamesra created GEODE-2874: -- Summary: StringIndexOutOfBoundsException while initializing logger Key: GEODE-2874 URL: https://issues.apache.org/jira/browse/GEODE-2874 Project: Geode

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-03 Thread Kirk Lund
> On May 3, 2017, 8:34 p.m., Ken Howe wrote: > > geode-core/src/main/java/org/apache/geode/cache/query/internal/index/FunctionalIndexCreationHelper.java > > Line 105 (original), 111 (patched) > > > > > > typo?

Re: [gemfire-dev] New Client-Server Protocol Proposal

2017-05-03 Thread Bruce Schuchardt
I agree with Dan that the spec will need to deal with the effects of retrying an operation. Le 5/3/2017 à 10:58 AM, Hitesh Khamesra a écrit : (Sorry: one more attempt to format this message) Here are the few things we need to consider.. 1. key, value, callbackarg can be required to

[jira] [Commented] (GEODE-2847) Retried bulk operation from client could not get applied version tag from the server and cause data inconsistency

2017-05-03 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2847?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15995662#comment-15995662 ] ASF subversion and git services commented on GEODE-2847: Commit

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

2017-05-03 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2828?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15995660#comment-15995660 ] ASF subversion and git services commented on GEODE-2828: Commit

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

2017-05-03 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2825?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15995654#comment-15995654 ] ASF subversion and git services commented on GEODE-2825: Commit

[jira] [Commented] (GEODE-2847) Retried bulk operation from client could not get applied version tag from the server and cause data inconsistency

2017-05-03 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2847?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15995661#comment-15995661 ] ASF subversion and git services commented on GEODE-2847: Commit

[jira] [Commented] (GEODE-2843) User Guide - example should specify

2017-05-03 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2843?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15995652#comment-15995652 ] ASF subversion and git services commented on GEODE-2843: Commit

[jira] [Commented] (GEODE-2843) User Guide - example should specify

2017-05-03 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2843?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15995659#comment-15995659 ] ASF subversion and git services commented on GEODE-2843: Commit

[jira] [Commented] (GEODE-2843) User Guide - example should specify

2017-05-03 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2843?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15995656#comment-15995656 ] ASF subversion and git services commented on GEODE-2843: Commit

[jira] [Commented] (GEODE-2870) BucketMovedException during function execution may lead to client missing results

2017-05-03 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2870?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15995620#comment-15995620 ] ASF subversion and git services commented on GEODE-2870: Commit

Re: [gemfire-dev] New Client-Server Protocol Proposal

2017-05-03 Thread Michael Stolz
The TCP fragmentation is fine for what it is, but it is *not* paging, and paging has long been something that we have wanted to get around to. -- Mike Stolz Principal Engineer, GemFire Product Manager Mobile: +1-631-835-4771 On Wed, May 3, 2017 at 1:33 PM, Galen M O'Sullivan

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-03 Thread Ken Howe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5/#review173809 ---

Re: [gemfire-dev] New Client-Server Protocol Proposal

2017-05-03 Thread Galen M O'Sullivan
On Tue, May 2, 2017 at 11:52 AM, Hitesh Khamesra < hitesh...@yahoo.com.invalid> wrote: > Absolutely its a implementation detail. > This doesn't answer Dan's comment. Do you think fragmentation should be taken care of by the TCP layer or the protocol should deal with it specifically?

[jira] [Resolved] (GEODE-2873) IllegalStateException when exception has non-ASCII characters.

2017-05-03 Thread Jacob S. Barrett (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2873?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jacob S. Barrett resolved GEODE-2873. - Resolution: Fixed > IllegalStateException when exception has non-ASCII characters. >

[jira] [Commented] (GEODE-2873) IllegalStateException when exception has non-ASCII characters.

2017-05-03 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15995570#comment-15995570 ] ASF subversion and git services commented on GEODE-2873: Commit

Broken: apache/geode#2486 (develop - 0dd2552)

2017-05-03 Thread Travis CI
Build Update for apache/geode - Build: #2486 Status: Broken Duration: 5 minutes and 57 seconds Commit: 0dd2552 (develop) Author: eshu Message: GEODE-2847: Get correct version tags for retried bulk operations Get correct version tags from

Fixed: apache/geode#2487 (develop - 1683265)

2017-05-03 Thread Travis CI
Build Update for apache/geode - Build: #2487 Status: Fixed Duration: 16 minutes and 34 seconds Commit: 1683265 (develop) Author: eshu Message: GEODE-2847: Get correct version tags for retried bulk operation View the changeset:

[jira] [Created] (GEODE-2873) IllegalStateException when exception has non-ASCII characters.

2017-05-03 Thread Jacob S. Barrett (JIRA)
Jacob S. Barrett created GEODE-2873: --- Summary: IllegalStateException when exception has non-ASCII characters. Key: GEODE-2873 URL: https://issues.apache.org/jira/browse/GEODE-2873 Project: Geode

[jira] [Created] (GEODE-2872) totalHitCount and totalMissCount incorrect for Partitioned Regions

2017-05-03 Thread Srikanth Manvi (JIRA)
Srikanth Manvi created GEODE-2872: - Summary: totalHitCount and totalMissCount incorrect for Partitioned Regions Key: GEODE-2872 URL: https://issues.apache.org/jira/browse/GEODE-2872 Project: Geode

Re: can cache-1.0.xsd be modified?

2017-05-03 Thread John Blum
I should clarify, if a feature is big enough, or there are many new additions, then I might bump the version and include the changes in the next release. However, to be clear, I don't think this is one of those changes. Additionally, I handle schema (XSD) versioning a little bit differently in

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-03 Thread Jared Stewart
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5/#review173759 ---

[jira] [Created] (GEODE-2871) CI Failure: org.apache.geode.management.internal.cli.commands.CommandOverHttpDUnitTest#testAlterUpdatesSharedConfig

2017-05-03 Thread Jared Stewart (JIRA)
Jared Stewart created GEODE-2871: Summary: CI Failure: org.apache.geode.management.internal.cli.commands.CommandOverHttpDUnitTest#testAlterUpdatesSharedConfig Key: GEODE-2871 URL:

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-03 Thread Ken Howe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5/#review173786 ---

[jira] [Updated] (GEODE-2870) BucketMovedException during function execution may lead to client missing results

2017-05-03 Thread Jason Huynh (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2870?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jason Huynh updated GEODE-2870: --- Affects Version/s: (was: 1.0.0-incubating.M1) 1.1.0 > BucketMovedException

[jira] [Updated] (GEODE-2870) BucketMovedException during function execution may lead to client missing results

2017-05-03 Thread Jason Huynh (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2870?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jason Huynh updated GEODE-2870: --- Description: If a function isHA and hasResult, if checkForBucketMovement() throws the

[jira] [Updated] (GEODE-2870) BucketMovedException during function execution may lead to client missing results

2017-05-03 Thread Jason Huynh (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2870?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jason Huynh updated GEODE-2870: --- Affects Version/s: 1.0.0-incubating.M1 > BucketMovedException during function execution may lead to

[jira] [Assigned] (GEODE-1994) Change geode StringUtils to extend commons StringUtils

2017-05-03 Thread Patrick Rhomberg (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-1994?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Patrick Rhomberg reassigned GEODE-1994: --- Assignee: Patrick Rhomberg (was: Kirk Lund) > Change geode StringUtils to extend

[jira] [Created] (GEODE-2870) BucketMovedException during function execution may lead to client missing results

2017-05-03 Thread Jason Huynh (JIRA)
Jason Huynh created GEODE-2870: -- Summary: BucketMovedException during function execution may lead to client missing results Key: GEODE-2870 URL: https://issues.apache.org/jira/browse/GEODE-2870 Project:

[jira] [Assigned] (GEODE-2870) BucketMovedException during function execution may lead to client missing results

2017-05-03 Thread Jason Huynh (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2870?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jason Huynh reassigned GEODE-2870: -- Assignee: Jason Huynh > BucketMovedException during function execution may lead to client

Re: can cache-1.0.xsd be modified?

2017-05-03 Thread John Blum
Hi Bruce- The general rule of thumb I use in SDG is that an XSD may be changed iff it is only introducing something new and non-required. For example, a new element to add configuration support via XML that did not exist before, but is not required (e.g. configuring the snapshot service).

[jira] [Created] (GEODE-2869) CI Failure: org.apache.geode.management.internal.cli.commands.ConfigCommandsDUnitTest

2017-05-03 Thread Jared Stewart (JIRA)
Jared Stewart created GEODE-2869: Summary: CI Failure: org.apache.geode.management.internal.cli.commands.ConfigCommandsDUnitTest Key: GEODE-2869 URL: https://issues.apache.org/jira/browse/GEODE-2869

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-03 Thread Kirk Lund
> On May 3, 2017, 5:19 p.m., Jared Stewart wrote: > > geode-core/src/main/java/org/apache/geode/internal/cache/PartitionAttributesImpl.java > > Line 770 (original), 766 (patched) > > > > > > I think this sort of

[jira] [Created] (GEODE-2868) CI Failure: org.apache.geode.management.internal.cli.commands.ConcurrentDeployDUnitTest#testMultipleGfshClientToOneServer

2017-05-03 Thread Jared Stewart (JIRA)
Jared Stewart created GEODE-2868: Summary: CI Failure: org.apache.geode.management.internal.cli.commands.ConcurrentDeployDUnitTest#testMultipleGfshClientToOneServer Key: GEODE-2868 URL:

[jira] [Commented] (GEODE-2788) Add official Socket timeout parameter when connecting to servers/locators

2017-05-03 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2788?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15995383#comment-15995383 ] ASF GitHub Bot commented on GEODE-2788: --- Github user bschuchardt commented on a diff in the pull

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-03 Thread Kirk Lund
> On May 3, 2017, 5:13 p.m., Jared Stewart wrote: > > geode-core/src/main/java/org/apache/geode/internal/cache/Oplog.java > > Line 125 (original), 125 (patched) > > > > > > Why do we want to make this class

[GitHub] geode pull request #474: GEODE-2788: Add official Socket timeout parameter w...

2017-05-03 Thread bschuchardt
Github user bschuchardt commented on a diff in the pull request: https://github.com/apache/geode/pull/474#discussion_r114618650 --- Diff: geode-core/src/main/resources/META-INF/schemas/geode.apache.org/schema/cache/cache-1.0.xsd --- @@ -1191,6 +1191,7 @@ As of 6.5 disk-dirs is

[jira] [Created] (GEODE-2867) CI Failure: org.apache.geode.management.LocatorManagementDUnitTest

2017-05-03 Thread Jared Stewart (JIRA)
Jared Stewart created GEODE-2867: Summary: CI Failure: org.apache.geode.management.LocatorManagementDUnitTest Key: GEODE-2867 URL: https://issues.apache.org/jira/browse/GEODE-2867 Project: Geode

Re: Review Request 58888: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache

2017-05-03 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5/#review173688 ---

  1   2   >