Re: WIKI migration is almost done. Volunteers needed, though

2015-05-04 Thread Barry Oglesby
Hi Roman, My confluence id is barry.oglesby. Can you enable me to edit? Thanks, Barry Oglesby Enterprise Architect For prompt responses to questions on GemFire/SQLFire, please email rtds-dev-ea at gopivotal dot com On Sun, May 3, 2015 at 6:04 PM, Roman Shaposhnik wrote: > Hi! > >

Re: Our JIRA is fully functional now

2015-05-05 Thread Barry Oglesby
barry.oglesby Barry Oglesby Enterprise Architect For prompt responses to questions on GemFire/SQLFire, please email rtds-dev-ea at gopivotal dot com On Tue, May 5, 2015 at 4:35 PM, Roman Shaposhnik wrote: > https://issues.apache.org/jira/browse/GEODE-1 > > Please make sure to register

Re: Native client connections

2015-07-01 Thread Barry Oglesby
listens on just that address: [info 2015/07/01 17:30:56.292 PDT tid=0x1] Bridge server connection listener bound to address /xxx.xxx.x.xx:40404 with backlog 1,000. Based on that test, there must be a case where setting the bind-address still causes the connection listener to listen on all local

Re: PartitionListener

2015-08-27 Thread Barry Oglesby
There are lots of customer using PartitionListener. The main use cases are to maintain local in-memory or disk-based state (like lucene indexes). Barry Oglesby GemFire Advanced Customer Engineering (ACE) For immediate support please contact Pivotal Support at http://support.pivotal.io/ On Wed

Re: PartitionManager

2015-08-27 Thread Barry Oglesby
I know of one customer using PartitionManager along with PartitionListener exactly like the javadoc comment in the class. Barry Oglesby GemFire Advanced Customer Engineering (ACE) For immediate support please contact Pivotal Support at http://support.pivotal.io/ On Wed, Aug 26, 2015 at 4:33 PM

Re: Create Region API

2015-09-14 Thread Barry Oglesby
t;> ] And add it to the cluster.xml file. The null in the addXmlEntity is a String[] of groups, so to add the region to specific group(s), set them in that parameter. Barry Oglesby GemFire Advanced Customer Engineering (ACE) For immediate support please contact Pivotal Support at ht

Re: Create Region API

2015-09-14 Thread Barry Oglesby
egion. I can probably dig up an example of this if you want one. Barry Oglesby GemFire Advanced Customer Engineering (ACE) For immediate support please contact Pivotal Support at http://support.pivotal.io/ On Mon, Sep 14, 2015 at 12:39 PM, Barry Oglesby wrote: > You can do somethin

Re: Create Region API

2015-09-15 Thread Barry Oglesby
ion on all existing members. If you do care, then you'll need something to store the region definitions so that new members create the same regions. Barry Oglesby GemFire Advanced Customer Engineering (ACE) For immediate support please contact Pivotal Support at http://support.pivotal.io/ On Tu

Re: Review Request 38440: add thread pool for p2p reader threads and async close threads

2015-09-22 Thread Barry Oglesby
lds is there because closeTransientFields could have been called more than once for the same CacheClientProxy. - Barry Oglesby On Sept. 18, 2015, 9:43 p.m., Darrel Schneider wrote: > > --- > This is an automatically generated e-m

Re: AsyncEventListener - How to detect duplicate events

2015-09-23 Thread Barry Oglesby
duplicates. The getPossibleDuplicate method is the way to detect whether these events are possible duplicates. Barry Oglesby GemFire Advanced Customer Engineering (ACE) For immediate support please contact Pivotal Support at http://support.pivotal.io/ On Wed, Sep 23, 2015 at 1:09 AM, Nilkanth Patel

Re: Cluster Configuration Service Clarification

2015-09-25 Thread Barry Oglesby
Wes, The directory and files don't get created unless they need to be. Are you creating regions or anything using gfsh? The directory and files will get created at that time. Barry Oglesby GemFire Advanced Customer Engineering (ACE) For immediate support please contact Pivotal Support at

Re: Cluster Configuration Service Clarification

2015-09-25 Thread Barry Oglesby
Wes, I'm not sure about the "other commands", but the rest of what you said is right. The cluster config files are not created based on xml configuration, but only when gfsh is used. Barry Oglesby GemFire Advanced Customer Engineering (ACE) For immediate support please contact Piv

Re: Review Request 39279: GEODE-397: Fix clients to use server ssl config

2015-10-13 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39279/#review102492 --- Ship it! Ship It! - Barry Oglesby On Oct. 13, 2015, 5:07 p.m

Re: Review Request 39278: GEODE-396: Fix server to use server/gateway ssl config

2015-10-13 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39278/#review102490 --- Ship it! Ship It! - Barry Oglesby On Oct. 13, 2015, 4:44 p.m

Review Request 39295: GEODE-402: MembershipListener callbacks are not always invoked

2015-10-13 Thread Barry Oglesby
36d160248647b5a25bf1e82e8d7709dfb84d0261 Diff: https://reviews.apache.org/r/39295/diff/ Testing --- Thanks, Barry Oglesby

Re: Review Request 39295: GEODE-402: MembershipListener callbacks are not always invoked

2015-10-13 Thread Barry Oglesby
n automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39295/#review102551 ------- On Oct. 13, 2015, 10:26 p.m., Barry Oglesby wrote: > > --- > This is an automatically genera

Review Request 39303: GEODE-402: MembershipListener callbacks are not always invoked

2015-10-13 Thread Barry Oglesby
exception handling for all callbacks Refactored handler code Diffs - gemfire-core/src/main/java/com/gemstone/gemfire/distributed/internal/DistributionManager.java 36d160248647b5a25bf1e82e8d7709dfb84d0261 Diff: https://reviews.apache.org/r/39303/diff/ Testing --- Thanks, Barry

Re: Review Request 40164: GEODE-541: The latest GemFire 800x clients can't connect to Geode servers

2015-11-10 Thread Barry Oglesby
https://reviews.apache.org/r/40164/diff/ Testing --- hand-tested GemFire 8009 client with Geode servers before and after change: - connected to locator - did puts - executed region function (both single-hop and non-single-hop - executed server function Thanks, Barry Oglesby

Re: Review Request 40272: GEODE-546: Resetting stats when AbstractGatewaySender is stopped

2015-11-13 Thread Barry Oglesby
enough? - Barry Oglesby On Nov. 13, 2015, 1:45 a.m., Dan Smith wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 40722: GEODE-606: Add some logging to war file detection

2015-12-01 Thread Barry Oglesby
be (where you looked for them) in the warning? - Barry Oglesby On Nov. 25, 2015, 10:55 p.m., Jens Deppe wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 40722: GEODE-606: Add some logging to war file detection

2015-12-01 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40722/#review108542 --- Ship it! Ship It! - Barry Oglesby On Dec. 1, 2015, 5:42 p.m

Re: Review Request 40871: GEODE-184: The gfsh 'locate entry' command fails to find the entry on partitioned regions if the key is not a string

2015-12-02 Thread Barry Oglesby
To reply, visit: > https://reviews.apache.org/r/40871/ > --- > > (Updated Dec. 2, 2015, 4:13 p.m.) > > > Review request for geode, Barry Oglesby and Darrel Schneider. > > > Repository: geode > > > Description > --- > > GEOD

Re: Review Request 40871: GEODE-184: The gfsh 'locate entry' command fails to find the entry on partitioned regions if the key is not a string

2015-12-02 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40871/#review108684 --- Ship it! Ship It! - Barry Oglesby On Dec. 2, 2015, 5:40 p.m

Re: Question on Querying and PDX Serialization

2015-12-03 Thread Barry Oglesby
0 Rows : 1 Result -- 1 gfsh>query --query="SELECT * FROM /data WHERE id='1'" Result : true startCount : 0 endCount : 20 Rows : 1 shares | price | id | cusip -- | | -- | - 22 | 329.297607421875 | 1 | HMIN Barry Ogl

Re: Function service usability improvements

2015-12-17 Thread Barry Oglesby
These look good. One small change I would like to see: Can you make these consistent (either withArguments or getArgs) Execution: public Execution withArgs(Object args); FunctionContext: public Object getArguments(); Barry Oglesby GemFire Advanced Customer Engineering (ACE) For immediate

Re: Function service usability improvements

2015-12-17 Thread Barry Oglesby
I'd rather have it spelled out with getArguments / setArguments. Thanks, Barry Oglesby GemFire Advanced Customer Engineering (ACE) For immediate support please contact Pivotal Support at http://support.pivotal.io/ On Thu, Dec 17, 2015 at 2:46 PM, Dan Smith wrote: > Hi Barry, > &

Re: Client/Server?

2016-01-20 Thread Barry Oglesby
ed by the client to determine how many servers there are and their locations. I can point you to that if you're interested. Barry Oglesby GemFire Advanced Customer Engineering (ACE) For immediate support please contact Pivotal Support at http://support.pivotal.io/ On Wed, Jan 20, 2016 at 3:2

Re: Client/Server?

2016-01-21 Thread Barry Oglesby
); } } If I start the TestClient before the server starts, it waits in waitForServer. Once the server starts, it gets notified and continues. If I start the TestClient after the server starts, it just sails through the waitForServer. Barry Oglesby GemFire Advanced Customer Engineering (ACE) F

Review Request 42664: GEODE-825: A client with subscriptions enabled receives an additional event when invoking get with a CacheLoader

2016-01-22 Thread Barry Oglesby
/InterestListDUnitTest.java ac4b92e9eed128ca3507b27dec410b670e676ebc Diff: https://reviews.apache.org/r/42664/diff/ Testing --- precheckin new InterestListDUnitTest cases Thanks, Barry Oglesby

Re: Build time and time per task

2016-01-22 Thread Barry Oglesby
talgf-assembly:compileTestJava 0.490s There is also a TaskExecutionListener that you can implement and configure, although I haven't tried that. Barry Oglesby GemFire Advanced Customer Engineering (ACE) For immediate support please contact Pivotal Support at http://support.pivotal.io/ On Fri, Jan 22, 2016 a

Re: Review Request 42822: GEODE-864: Waiting for conflation thread in conflation tests

2016-01-26 Thread Barry Oglesby
ot;ln" )); I'm surprised you left the 'KBKBKB' log messages in there. :-) - Barry Oglesby On Jan. 26, 2016, 8:47 p.m., Dan Smith wrote: > > --- > This is an autom

Review Request 42833: GEODE-865: CI Failure: ConcurrentParallelGatewaySenderOperation_2_DUnitTest.testParallelGatewaySenders_SingleNode_UserPR_localDestroy_RecreateRegion

2016-01-26 Thread Barry Oglesby
- gemfire-wan/src/test/java/com/gemstone/gemfire/internal/cache/wan/concurrent/ConcurrentParallelGatewaySenderOperation_2_DUnitTest.java f770062e2a2fdb8c9d35cd2d4f76ac96fa56ac37 Diff: https://reviews.apache.org/r/42833/diff/ Testing --- Thanks, Barry Oglesby

Review Request 42889: GEODE-865: CI Failure: ConcurrentParallelGatewaySenderOperation_2_DUnitTest.testParallelGatewaySenders_SingleNode_UserPR_localDestroy_RecreateRegion

2016-01-27 Thread Barry Oglesby
: https://reviews.apache.org/r/42889/diff/ Testing --- Thanks, Barry Oglesby

Re: Review Request 42889: GEODE-865: CI Failure: ConcurrentParallelGatewaySenderOperation_2_DUnitTest.testParallelGatewaySenders_SingleNode_UserPR_localDestroy_RecreateRegion

2016-01-27 Thread Barry Oglesby
times in other files too. I'll move it up to WANTestBase. - Barry --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42889/#review116686 --------

Re: Review Request 42892: GEODE-873: Allowing for possible retry in testParallelPregationHA

2016-01-27 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42892/#review116690 --- Ship it! Ship It! - Barry Oglesby On Jan. 28, 2016, 1:47

Re: Review Request 42885: GEODE-872: Removing duplicate code from ConcurrentSerialGatewaySenderOperationsDUnitTest

2016-01-27 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42885/#review116691 --- Ship it! Ship It! - Barry Oglesby On Jan. 28, 2016, 12:43

Re: Review Request 42886: GEODE-867: Ignore secondaries in waitForDispatcherToPause

2016-01-27 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42886/#review116693 --- Ship it! Ship It! - Barry Oglesby On Jan. 28, 2016, 12:46

Re: Listener/Writer events for Region.get() operation

2016-01-28 Thread Barry Oglesby
includes a readme that describes it. Barry Oglesby GemFire Advanced Customer Engineering (ACE) For immediate support please contact Pivotal Support at http://support.pivotal.io/ On Thu, Jan 28, 2016 at 6:43 AM, Nilkanth Patel wrote: > Hello, > > Is it possible to get beforeGet(), af

Re: [VOTE] RC2: Apache Geode (Incubating) first Milestone release - v1.0.0-incubating.M1

2016-01-28 Thread Barry Oglesby
+1 Barry Oglesby GemFire Advanced Customer Engineering (ACE) For immediate support please contact Pivotal Support at http://support.pivotal.io/ On Thu, Jan 28, 2016 at 2:42 PM, Dan Smith wrote: > +1 > > I kicked some tires and looks good! We should prioritize fixing some of > thes

Review Request 42977: GEODE-875: CI failure: ParallelGatewaySenderOperationsOffHeapDUnitTest.testParallelGatewaySender_SingleNode_UserPR_localDestroy_SimultenuousPut_RecreateRegion

2016-01-29 Thread Barry Oglesby
PRE-CREATION gemfire-wan/src/test/java/com/gemstone/gemfire/internal/cache/wan/parallel/ParallelGatewaySenderOperationsDUnitTest.java 19f6c4bf559b9238f369aaa06c4b8594a3439497 Diff: https://reviews.apache.org/r/42977/diff/ Testing --- Thanks, Barry Oglesby

Re: Review Request 42941: Fix for GEODE-877 CacheXmlGenerator creates index xml with internal alias with expression.

2016-02-01 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42941/#review117228 --- Ship it! Ship It! - Barry Oglesby On Jan. 29, 2016, 2:09

Re: Review Request 42977: GEODE-875: CI failure: ParallelGatewaySenderOperationsOffHeapDUnitTest.testParallelGatewaySender_SingleNode_UserPR_localDestroy_SimultenuousPut_RecreateRegion

2016-02-01 Thread Barry Oglesby
This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42977/#review117038 ----------- On Jan. 29, 2016, 9:34 p.m., Barry Oglesby wrote: > > ---

Re: Review Request 43084: GEODE-892: Cleanup unnecessary compression code from wan test

2016-02-02 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43084/#review117420 --- Ship it! Ship It! - Barry Oglesby On Feb. 2, 2016, 5:07 a.m

Re: Review Request 43346: GEODE-657: Add an abort flag to the FinishBackupMessage

2016-02-09 Thread Barry Oglesby
still be in the middle of processing a backup? - Barry Oglesby On Feb. 9, 2016, 12:33 a.m., Dan Smith wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 43346: GEODE-657: Add an abort flag to the FinishBackupMessage

2016-02-11 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43346/#review118894 --- Ship it! Ship It! - Barry Oglesby On Feb. 10, 2016, 11:03

Review Request 43654: GEODE-967: Added xml support for GatewayEventSubstitutionFilter

2016-02-17 Thread Barry Oglesby
, Barry Oglesby

Review Request 43674: GEODE-969: Decreases WAN dunit log file size

2016-02-17 Thread Barry Oglesby
--- ParallelWANPersistenceEnabledGatewaySenderOffHeapDUnitTest Thanks, Barry Oglesby

Re: Review Request 43797: GEODE-478: Message class length field overflows if size is > 2GB

2016-02-19 Thread Barry Oglesby
writeException method? BaseCommand protected static void writeException(Message origMsg, int msgType, Throwable e, ... if (e instanceof MessageTooLargeException) { throw (IOException)e; } - Barry Oglesby On Feb. 20, 2016, 12:43 a.m., Bruce Schuchardt wrote

Re: Review Request 43797: GEODE-478: Message class length field overflows if size is > 2GB

2016-02-19 Thread Barry Oglesby
my changes from 820X_maint. - Barry Oglesby On Feb. 20, 2016, 12:43 a.m., Bruce Schuchardt wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 43797: GEODE-478: Message class length field overflows if size is > 2GB

2016-02-22 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43797/#review120164 --- Ship it! Ship It! - Barry Oglesby On Feb. 20, 2016, 12:43

Re: Review Request 44929: GEODE-1067: The dispatcher now handles IllegalStateException by retrying batch without the released event

2016-03-18 Thread Barry Oglesby
ze() != 0 && keepWaiting) { continue; } - Barry --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44929/#review124048 ----------

Review Request 44928: GEODE-670: GatewaySenderEvents are now sized using Sizeable if appropriate

2016-03-19 Thread Barry Oglesby
/cache/BucketRegion.java 413fc876c3493b51be0f6d61326d6881ddaee8ce Diff: https://reviews.apache.org/r/44928/diff/ Testing --- - repeatedly ran ParallelGatewaySenderOperation_2_OffHeapDUnitTest (along fixes for GEODE-1067 and GEODE-1095) - running precheckin Thanks, Barry Oglesby

Review Request 44930: GEODE-1095: GatewaySenderEventImpl now caches serialized value size

2016-03-19 Thread Barry Oglesby
Diff: https://reviews.apache.org/r/44930/diff/ Testing --- - repeatedly ran ParallelGatewaySenderOperation_2_OffHeapDUnitTest (along fixes for GEODE-670 and GEODE-1067) - running precheckin Thanks, Barry Oglesby

Review Request 44929: GEODE-1067: The dispatcher now handles IllegalStateException by retrying batch without the released event

2016-03-20 Thread Barry Oglesby
://reviews.apache.org/r/44929/diff/ Testing --- - repeatedly ran ParallelGatewaySenderOperation_2_OffHeapDUnitTest (along fixes for GEODE-670 and GEODE-1095) - running precheckin Thanks, Barry Oglesby

Re: Review Request 45162: GEODE-911: Cleaning up SerialGatewaySender queues when stopping gateway

2016-03-22 Thread Barry Oglesby
Set queues = getQueues(); if (queues != null && !queues.isEmpty()) { for (RegionQueue q : queues) { ((SerialGatewaySenderQueue)q).cleanUp(); } } Also, SerialGatewaySenderOperationsDUnitTest validateQueueClosed isn't implemented. Is that intentional? - Barry O

Re: Review Request 45180: GEODE-1054: InternalResultSender.setException logging at info level instead of fatal

2016-03-22 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45180/#review124913 --- Ship it! Ship It! - Barry Oglesby On March 22, 2016, 10:37

Re: Review Request 45162: GEODE-911: Cleaning up SerialGatewaySender queues when stopping gateway

2016-03-22 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45162/#review124914 --- Ship it! Ship It! - Barry Oglesby On March 22, 2016, 10:29

Re: Review Request 45224: GEODE-1108: Ensure that the maximum message size gets reset

2016-03-23 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45224/#review125078 --- Ship it! Ship It! - Barry Oglesby On March 23, 2016, 5:53

Re: Review Request 45227: GEODE-988: CI Failure: CqPerfUsingPoolDUnitTest.testMatchingCqs failed with AssertionFailedError

2016-03-23 Thread Barry Oglesby
/CqQueryUsingPoolDUnitTest.java (line 1020) <https://reviews.apache.org/r/45227/#comment187849> I assume you meant to comment out all the logging? If so, you might as well just delete the calls. - Barry Oglesby On March 23, 2016, 6:03 p.m., anilkumar gingade

Re: Review Request 45227: GEODE-988: CI Failure: CqPerfUsingPoolDUnitTest.testMatchingCqs failed with AssertionFailedError

2016-03-23 Thread Barry Oglesby
/CqQueryUsingPoolDUnitTest.java (line 1009) <https://reviews.apache.org/r/45227/#comment187848> I'm not sure how this is validating all the listeners. It still looks like it is validating just the first one. Is that what you want? - Barry Oglesby On March 23, 2016, 6:03 p.m.

Re: Review Request 45346: GEODE-478: Adding a stat for resized gateway batches

2016-03-25 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45346/#review125487 --- Ship it! Ship It! - Barry Oglesby On March 25, 2016, 11:02

Re: Review Request 45709: GEODE-1062: removing similar/duplicate create receiver methods

2016-04-04 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45709/#review126989 --- Ship it! Ship It! - Barry Oglesby On April 4, 2016, 10:14

Review Request 45787: GEODE-1170: The GatewaySender for the AsyncEventQueue is now marked internal

2016-04-05 Thread Barry Oglesby
/45787/diff/ Testing --- - precheckin - customer use case Thanks, Barry Oglesby

Review Request 45788: GEODE-1172: Closed all GatewaySenders when a ForcedDisconnectException occurs

2016-04-05 Thread Barry Oglesby
5669aedf1f812f453cc5bfd349db45c23a22afca geode-core/src/main/java/com/gemstone/gemfire/internal/cache/GemFireCacheImpl.java 9902be50095c2b67b8121d7f7e45566d2334eef1 Diff: https://reviews.apache.org/r/45788/diff/ Testing --- - precheckin - customer use case Thanks, Barry Oglesby

Re: Review Request 45938: GEODE-1062: Undoing reduction in wait time

2016-04-08 Thread Barry Oglesby
might as well be forever. - Barry Oglesby On April 8, 2016, 6:37 p.m., Jason Huynh wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 45938: GEODE-1062: Undoing reduction in wait time

2016-04-08 Thread Barry Oglesby
> On April 8, 2016, 8:02 p.m., Barry Oglesby wrote: > > Maybe that timeout should be configurable? Most times 4 minutes might as > > well be forever. > > Jason Huynh wrote: > Yes, that would be a good idea. I'm trying to revert this one change in > the b

Re: Review Request 45938: GEODE-1062: Undoing reduction in wait time

2016-04-08 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45938/#review127870 --- Ship it! Ship It! - Barry Oglesby On April 8, 2016, 6:37

Re: Review Request 45938: GEODE-1062: Undoing reduction in wait time

2016-04-08 Thread Barry Oglesby
> On April 8, 2016, 8:02 p.m., Barry Oglesby wrote: > > Maybe that timeout should be configurable? Most times 4 minutes might as > > well be forever. > > Jason Huynh wrote: > Yes, that would be a good idea. I'm trying to revert this one change in > the b

Re: Review Request 45938: GEODE-1062: Undoing reduction in wait time

2016-04-08 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45938/#review127879 --- Ship it! Ship It! - Barry Oglesby On April 8, 2016, 8:26

Re: Review Request 46154: GEODE-1018: Retrieve queues only once gateway listener to avoid a race

2016-04-13 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46154/#review128718 --- Ship it! Ship It! - Barry Oglesby On April 13, 2016, 5:18

Re: Review Request 46914: GEODE-11: Fixing a class cast exception when LuceneFunction has an error

2016-05-02 Thread Barry Oglesby
est guy to review the test changes. - Barry Oglesby On May 2, 2016, 10:32 p.m., Dan Smith wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://revi

Re: Broken: apache/incubator-geode#451 (develop - 2a786ee)

2016-05-02 Thread Barry Oglesby
I fixed this issue. Thanks, Barry Oglesby On Mon, May 2, 2016 at 2:34 PM, Travis CI wrote: > Build Update for apache/incubator-geode > - > > Build: #451 > Status: Broken > > Duration: 15 minutes and 0 seconds > Commit: 2a786ee (

Re: Review Request 46988: GEODE-1330: Set boolean so that internal lucene regions aren't generated in xml

2016-05-09 Thread Barry Oglesby
ION Diff: https://reviews.apache.org/r/46988/diff/ Testing --- - lucene precheckin Thanks, Barry Oglesby

Re: Review Request 47148: GEODE-11: Adding package level javadocs for lucene

2016-05-09 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47148/#review132363 --- Ship it! Ship It! - Barry Oglesby On May 10, 2016, 1:13 a.m

Re: Review Request 47326: GEODE-11: Adding tests of fixed partitions and persistent lucene indexes

2016-05-13 Thread Barry Oglesby
to creating the default disk store in the cache for every test run? - Barry Oglesby On May 13, 2016, 12:13 a.m., Dan Smith wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 47372: GEODE-1209: Added gfsh support to set ignoreEvictionAndExpiration attribute for create AEQ command.

2016-05-16 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47372/#review133385 --- Ship it! Ship It! - Barry Oglesby On May 13, 2016, 11:05

Re: Review Request 47572: GEODE-1352: Clean up cache in LuceneQueryImplIntegrationTest

2016-05-18 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47572/#review133856 --- Ship it! Ship It! - Barry Oglesby On May 19, 2016, 12:41

Re: Review Request 47496: Adding transaction tests for lucene indexes

2016-05-19 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47496/#review134034 --- Ship it! Ship It! - Barry Oglesby On May 19, 2016, 6:32 p.m

Re: Stats command

2016-05-23 Thread Barry Oglesby
primaryBucketCount max=22 Thanks, Barry Oglesby On Mon, May 23, 2016 at 9:30 AM, Darrel Schneider wrote: > I don't know of a way to analyze offline stat archives using gfsh. > > But the code for the old gemfire stats command is still in the geode >

Re: Review Request 47911: GEODE-11: Adding a test of queries with redundancy

2016-05-26 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47911/#review135033 --- Ship it! Ship It! - Barry Oglesby On May 26, 2016, 6:22 p.m

Re: Async queue mechanism

2016-05-31 Thread Barry Oglesby
I see this same behavior. I'll look into it a bit more and let you know what I find. Thanks, Barry Oglesby On Sun, May 29, 2016 at 5:38 AM, Gal Palmery wrote: > +Adding the dev group > > > > Thanks, > > Gal > > > > *From:* Gal Palmery > *Se

Re: Async queue mechanism

2016-05-31 Thread Barry Oglesby
I see the issue. It looks like a merge issue. I have a fix that I'll check in in the next day or so. Thanks, Barry Oglesby On Tue, May 31, 2016 at 12:52 PM, Barry Oglesby wrote: > I see this same behavior. I'll look into it a bit more and let you know > what I find. >

Re: Async queue mechanism

2016-06-01 Thread Barry Oglesby
Sure. I created https://issues.apache.org/jira/browse/GEODE-1471 last night and just added Gal's details this morning. Thanks, Barry Oglesby On Wed, Jun 1, 2016 at 3:44 AM, Dor Ben Dov wrote: > Barry, > > Can you open a jira issue to file this problem, maybe with Gal's

Re: Async queue mechanism

2016-06-01 Thread Barry Oglesby
left behind and not set on the AsyncEventQueueImpl. The fix was to not leave any configured GatewayEventFilters behind. Thanks, Barry Oglesby On Tue, May 31, 2016 at 11:16 PM, Gal Palmery wrote: > Thanks a lot Barry! > > Can you pls share the quick explanation about the fix? > &g

Re: Review Request 49966: GEODE-11 Adding tests of closing a cache during index updates

2016-07-12 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49966/#review141983 --- Ship it! Ship It! - Barry Oglesby On July 12, 2016, 9:18

Re: use-cluster-configuration/cache.xml/reconnect

2016-07-25 Thread Barry Oglesby
exact configuration is being used that is failing? Thanks, Barry Oglesby On Mon, Jul 25, 2016 at 10:44 AM, Kirk Lund wrote: > Below came to me via support engineer, and I'm trying to determine what the > correct behavior should be: > > "Let's assume that you&

Re: use-cluster-configuration/cache.xml/reconnect

2016-07-25 Thread Barry Oglesby
if use-cluster-configuration=false (in JGroupMembershipManager saveCacheXmlForReconnect) - if the cache-xml-file is set (in GemFireCacheImpl logCacheXML) Neither of these are true in the spring-only configuration, so nothing is reloaded. Thanks, Barry Oglesby On Mon, Jul 25, 2016 at 1:05 PM, K

Re: Review Request 50680: fix replace on overflowed entry

2016-08-01 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50680/#review144405 --- Ship it! Ship It! - Barry Oglesby On Aug. 1, 2016, 11:17

Re: Review Request 50680: fix replace on overflowed entry

2016-08-03 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50680/#review144660 --- Ship it! Ship It! - Barry Oglesby On Aug. 3, 2016, 6:36 p.m

Re: [VOTE] Release Apache Geode (incubating) 1.0.0-incubating.M3 - RC7

2016-08-08 Thread Barry Oglesby
+1 Thanks, Barry Oglesby On Mon, Aug 8, 2016 at 4:08 PM, William Markito wrote: > Guys, I'll leave the voting open until tomorrow, but please vote so we can > close and release M3. > > Thanks, > > On Fri, Aug 5, 2016 at 4:24 PM, Anthony Baker wrote: > >

Re: Review Request 51186: GEODE-1760, GEODE-1762: Tests of function exceptions from clients

2016-08-17 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51186/#review146031 --- Ship it! Ship It! - Barry Oglesby On Aug. 17, 2016, 5:37

Re: Review Request 51187: GEODE-1760 Sending partial results to user's ResultCollector

2016-08-17 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51187/#review146032 --- Ship it! Ship It! - Barry Oglesby On Aug. 17, 2016, 5:41

Re: Review Request 51188: GEODE-1776: Always read all of the results in ExecuteRegionFunctionOp

2016-08-17 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51188/#review146033 --- Ship it! Ship It! - Barry Oglesby On Aug. 17, 2016, 5:41

Re: Review Request 51305: GEODE-1810: WAN locator exchange doesn't include hostname-for-clients if it is set

2016-08-22 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51305/#review146458 --- Ship it! Ship It! - Barry Oglesby On Aug. 22, 2016, 10:42

Re: How to get an Mbean of client federated by JMX Manager?

2016-09-13 Thread Barry Oglesby
either using a region like this or a function. Either way, you probably want a server MBean providing this data, not a client MBean. Thanks, Barry Oglesby On Tue, Sep 13, 2016 at 4:48 PM, Jinmei Liao wrote: > It looks like you want to register an MBean in a remote MBeanServer. First > of al

Re: Review Request 52126: GEODE-1915: Prevent deadlock registering instantiators with gateways

2016-09-21 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52126/#review149861 --- Ship it! Ship It! - Barry Oglesby On Sept. 21, 2016, 5:40

Re: Statistics sampler is not available

2016-09-26 Thread Barry Oglesby
ion in the HostStatSampler run method should log something in the server log, though. Do you have a full server log file that you can post? Thanks, Barry Oglesby On Mon, Sep 26, 2016 at 3:10 PM, sri man wrote: > Hi, > > I am trying to experiment with exposing Geode stats to Grafana > <htt

Re: Review Request 53444: GEODE-2068 Improve documentation on serial/parallel gateway senders

2016-11-04 Thread Barry Oglesby
/multisite_overview.html.md.erb (line 68) <https://reviews.apache.org/r/53444/#comment224720> Parallel senders are only supported on partitioned (not distributed) regions. - Barry Oglesby On Nov. 4, 2016, 12:11 a.m., Karen Miller

Re: Review Request 53814: GEODE-2100 Add new version of query client server messages

2016-11-16 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53814/#review156104 --- Ship it! Ship It! - Barry Oglesby On Nov. 16, 2016, 5:22