Re: [Suspected Spam] [VOTE] Apache Geode 1.14.4.RC1

2022-03-15 Thread Eric Shu
+1 Run end to end test cases against the build without seeing new issues. From: Donal Evans Sent: Tuesday, March 15, 2022 4:25 PM To: dev@geode.apache.org Subject: Re: [Suspected Spam] [VOTE] Apache Geode 1.14.4.RC1 +1 Confirmed that performance across a

Re: [Suspected Spam] [VOTE] Apache Geode 1.14.3.RC1

2022-01-21 Thread Eric Shu
+1 for 1.14.3.RC1 Build and run certain tests against it. -Eric From: Dick Cavender Sent: Friday, January 21, 2022 9:25 AM To: dev@geode.apache.org Subject: [Suspected Spam] [VOTE] Apache Geode 1.14.3.RC1 Hello Geode Dev Community, This is a release

Re: Proposal to add GEODE-8947 to blocker list

2021-02-17 Thread Eric Shu
+1 From: Donal Evans Sent: Tuesday, February 16, 2021 11:57 PM To: dev@geode.apache.org Subject: Re: Proposal to add GEODE-8947 to blocker list +1 I can confirm that the performance degradations I saw that led to me opening GEODE-8974 are no longer present

Re: [PROPOSAL] Backport GEODE-8475 to 1.13

2020-09-02 Thread Eric Shu
+1 From: Dick Cavender Sent: Wednesday, September 2, 2020 11:07 AM To: dev@geode.apache.org Subject: RE: [PROPOSAL] Backport GEODE-8475 to 1.13 +1 -Original Message- From: Anilkumar Gingade Sent: Wednesday, September 2, 2020 11:05 AM To:

Re: [PROPOSAL] Backport GEODE-8432 to 1.13

2020-08-19 Thread Eric Shu
+1 From: Jinmei Liao Sent: Wednesday, August 19, 2020 2:09 PM To: dev@geode.apache.org Subject: Re: [PROPOSAL] Backport GEODE-8432 to 1.13 +1 On 8/19/20, 10:25 AM, "Xiaojian Zhou" wrote: This problem also exists in 1.13.

[PROPOSAL] port GEODE-8323 changes to support/1.13 branch.

2020-07-28 Thread Eric Shu
This is to address HARegionQueue GII events not properly removed issue, which could potentially resurrect a strayed event in client cache. Regards, Eric

Re: Request for wiki permission

2020-07-08 Thread Eric Shu
Kirk Lund wrote: > I see what's missing! > > On Wed, Jul 8, 2020 at 1:04 PM Eric Shu wrote: > >> Here is what I have when I search the name with my profile: >> >> >> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fcwiki.apache.org%2Fconflue

Re: Request for wiki permission

2020-07-08 Thread Eric Shu
vmware.com%7C42904c454b304d2a248708d82366aa44%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637298271131613202sdata=dHngozyOCrQmZNTg%2FhebEUZxmVOleGDWPF3oQ2yytJc%3Dreserved=0. Please make sure you've created the account on that server first. Then I can add the edit permissions for Geode. Thanks, Kirk On Wed, Jul 8, 2020 at 9:

Re: [DISCUSS] RFC - Avoid the queueing of dropped events by the primary gateway sender when the gateway sender is stopped

2020-07-08 Thread Eric Shu
ected memory exhaustion. I really think the solution proposed makes the behavior of the gateway sender command more logical. Best regards, Alberto ____ From: Eric Shu Sent: Wednesday, July 8, 2020 7:32 PM To: dev@geode.apache.org Subject: Re: [DISCUSS] RFC - Avoid

Re: [DISCUSS] RFC - Avoid the queueing of dropped events by the primary gateway sender when the gateway sender is stopped

2020-07-08 Thread Eric Shu
It seems that I was not able to comment on the RFC in the wiki yet. Just try to find out if we have a simple solution for the issue you raised -- can we have a up-limit for the tmpDroppedEvents queue in question? Always check the limit before adding to the queue -- so that the tmp queue is not

Request for wiki permission

2020-07-08 Thread Eric Shu
Hi, I am trying to comment on a wiki doc and need permissions for Confluence. Please grant it. UserName: eshu Regards, Eric

Re: [PROPOSAL] backport GEODE-8238 to support/1.13

2020-06-11 Thread Eric Shu
+1 From: Owen Nichols Sent: Thursday, June 11, 2020 10:58 AM To: dev@geode.apache.org Subject: Re: [PROPOSAL] backport GEODE-8238 to support/1.13 +1 --- Sent from Workspace ONE

Re: [PROPOSAL] backport GEODE-8174 to 1.13 and 1.12

2020-05-26 Thread Eric Shu
+1 From: Udo Kohlmeyer Sent: Tuesday, May 26, 2020 2:12 PM To: geode Subject: [PROPOSAL] backport GEODE-8174 to 1.13 and 1.12 Hi there Geode Dev, I would like to request a back port of a critical issue in the JCAConnectionManager. This issue manifests itself

Re: [PROPOSAL]: GEODE-8071 to support/1.12 and support/1.13

2020-05-07 Thread Eric Shu
+1 On Thu, May 7, 2020 at 10:22 AM Jianxia Chen wrote: > +1 > > On Thu, May 7, 2020 at 7:47 AM Ju@N wrote: > > > Hello devs, > > > > I'd like to propose bringing GEODE-8071 [1] to the *support/1.12* and > > *support/1.13* branches. > > The bug was introduced in Geode 1.8.0 and, long story

Re: [PROPOSAL] include GEODE-8073 in Geode 1.13 support branch

2020-05-06 Thread Eric Shu
On Wed, May 6, 2020 at 11:40 AM Owen Nichols > > wrote: > > > > > > > +1 to fix this NPE on support/1.13 and also support/1.12 > > > > > > > > > On May 6, 2020, at 11:19 AM, Eric Shu wrote: > > > > > > > > > > GEODE-8073 > > > > > > > > > > > > > > -- > > > Cheers > > > > > > Jinmei > > > > > >

[PROPOSAL] include GEODE-8073 in Geode 1.13 support branch

2020-05-06 Thread Eric Shu
I'd like to propose bringing GEODE-8073 to the support/1.13 branch. The issue is not recently introduced, but recent changes make it easier to show up. I think this is a critical issue and would like this fix to be included. Eric

Re: Help needed to get my PR to pass the stressNewTest

2020-03-04 Thread Eric Shu
would be a failure? > > On Wed, Mar 4, 2020 at 1:28 PM Eric Shu wrote: > > > My PR (https://github.com/apache/geode/pull/4709) continue to fail in > > stressNewTest. I have been retrigger the test and all failed with same > > issue. I need help to find out how

Help needed to get my PR to pass the stressNewTest

2020-03-04 Thread Eric Shu
My PR (https://github.com/apache/geode/pull/4709) continue to fail in stressNewTest. I have been retrigger the test and all failed with same issue. I need help to find out how to get this PR to pass the target. Thanks! -Eric > Task :geode-assembly:repeatDistributedTest 15:59:04 Do not allow

Re: Getting UnsupportedOperationInTransactionException whem tried to access 2 partition regions in a transaction.

2020-01-06 Thread Eric Shu
If roleRegion do not have a lot of entries, can you make it a replicated one? On Mon, Jan 6, 2020 at 5:02 PM Eric Shu wrote: > Sorry, did not realize that it's a many to many relations. The above > mentioned colocation suggestion would not help due to the roles are in a > map (can no

Re: Getting UnsupportedOperationInTransactionException whem tried to access 2 partition regions in a transaction.

2020-01-06 Thread Eric Shu
Sorry, did not realize that it's a many to many relations. The above mentioned colocation suggestion would not help due to the roles are in a map (can not achieve the colocation). Regards, Eric On Mon, Jan 6, 2020 at 4:59 PM Eric Shu wrote: > Please note distributed transact

Re: Getting UnsupportedOperationInTransactionException whem tried to access 2 partition regions in a transaction.

2020-01-06 Thread Eric Shu
Please note distributed transaction is not a supported feature in Geode (not fully implemented). There was discussion before whether we should remove the code or throw exception when transaction is set to distributed. However, neither was being done. Currently in Geode, if you want to have a

Request GEODE-7537 fix to be cherry-picked into release 1.11

2019-12-11 Thread Eric Shu
Hello, GEODE-7537 was highlighted as blocking 1.11 release. This issue is being addressed now. I would like to have this fix to be merged into release 1.11. The commit sha is 3a3db82afdf0668e98ca49bcbdfa478e0ac94e10 Regards, Eric

Re: [DISCUSS] Shall we change the docs to indicate that default setting of enableLocalCache is false for client/server in the Tomcat module

2019-11-19 Thread Eric Shu
on the above mentioned pros and cons. User could override the setting in the context.xml if needed. Let's close this discussion. Regards, Eric On Tue, Nov 19, 2019 at 5:05 PM Eric Shu wrote: > Dear Geode, > > Here is the current documentation on enableLocalCache for default > configurati

[DISCUSS] Shall we change the docs to indicate that default setting of enableLocalCache is false for client/server in the Tomcat module

2019-11-19 Thread Eric Shu
Dear Geode, Here is the current documentation on enableLocalCache for default configuration in Tomcat module (Changing the Default Geode Configuration in the Tomcat Module )

Re: [DISCUSS] add GEODE-7079 to release/1.9.2

2019-10-04 Thread Eric Shu
+1 On Fri, Oct 4, 2019 at 11:38 AM Anilkumar Gingade wrote: > +1 > > On Fri, Oct 4, 2019 at 11:15 AM Juan José Ramos wrote: > > > +1 > > > > > > > > > > On Fri, Oct 4, 2019 at 6:39 PM Jens Deppe wrote: > > > > > On behalf of Juan I'm requesting approval to add GEODE-7079 to > > >

Re: [VOTE] Adding a lucene specific fix to release/1.10.0

2019-09-19 Thread Eric Shu
+1 On Thu, Sep 19, 2019 at 10:59 AM Benjamin Ross wrote: > +1 > > On Thu, Sep 19, 2019 at 10:50 AM Nabarun Nag wrote: > > > +1 > > > > On Thu, Sep 19, 2019 at 10:49 AM Xiaojian Zhou wrote: > > > > > I want to merge GEODE-7208, which is lucene specific fix > > > > > > The fix will enable

Re: Propose including GEODE-7085 in 1.10

2019-08-26 Thread Eric Shu
+1 On Mon, Aug 26, 2019 at 10:40 AM Alexander Murmann wrote: > +1 > > While it's not new, it's critical > > On Mon, Aug 26, 2019 at 10:38 AM Juan José Ramos > wrote: > > > +1 > > > > On Mon, Aug 26, 2019 at 6:36 PM Dan Smith wrote: > > > > > Hi, > > > > > > I'd like to propose including the

Re: Propose fix for 1.10 release: Export offline data command failed with EntryDestroyedException

2019-08-16 Thread Eric Shu
me. When you need to do an offline export, it’s usually > > > important. Not being able to export *all* the data might lead to data > > loss. > > > > > > Anthony > > > > > > > > > > On Aug 16, 2019, at 2:06 PM, Udo Kohlmeyer

Propose fix for 1.10 release: Export offline data command failed with EntryDestroyedException

2019-08-16 Thread Eric Shu
Hi, I'd like to include the following commit ( https://gitbox.apache.org/repos/asf?p=geode.git;h=aa33060) into Geode 1.10 release. The commit fixes the issue that a user tries to use export offline data to a snapshot file but failed. This issue exists from release 1.1.0. However, it is a

Re: Proposal to Include GEODE-7079 in 1.10.0

2019-08-15 Thread Eric Shu
+1 On Thu, Aug 15, 2019 at 9:54 AM John Blum wrote: > +1 > > On Thu, Aug 15, 2019 at 5:30 AM Ju@N wrote: > > > Hello team, > > > > I'd like to propose including the *fix [1]* for *GEODE-7079 [2]* in > release > > 1.10.0. > > Long story short: a *NullPointerException* can be continuously thrown

Re: "Output path is not specified for module" (IntelliJ)

2019-06-20 Thread Eric Shu
ode-core:test. > > I saw the options to ‘inherit project compile path’ but there were tons of > modules and I am not sure if I have to set it on all of them. > > If anyone else has build and test working with IntelliJ and knows what I > am doing wrong, please chime in. > > Thanks, &g

Re: "Output path is not specified for module" (IntelliJ)

2019-06-06 Thread Eric Shu
I worked around this issue by choosing Inherit project compile output path. On Thu, Jun 6, 2019 at 3:16 PM Jens Deppe wrote: > I have only experienced this when I switch to building with IntelliJ (it is > also dependent on what Intellij deems necessary to build). Building with > gradle has

GEODE-6630 fix for release 1.9.0

2019-04-18 Thread Eric Shu
I'd like to include the fix for the NPE. It is new in 1.9. Regards, Eric

Re: Review Request 62506: GEODE-3679: Original client member Id in the transaction should be set in the PartitionMessage

2017-09-22 Thread Eric Shu
ated e-mail. To reply, visit: https://reviews.apache.org/r/62506/#review186005 ------- On Sept. 22, 2017, 7:12 p.m., Eric Shu wrote: > > --- > This is a

Re: Review Request 62506: GEODE-3679: Original client member Id in the transaction should be set in the PartitionMessage

2017-09-22 Thread Eric Shu
d e-mail. To reply, visit: https://reviews.apache.org/r/62506/#review186003 ------- On Sept. 22, 2017, 7:12 p.m., Eric Shu wrote: > > --- > This is an automatically generat

Re: Review Request 62506: GEODE-3679: Original client member Id in the transaction should be set in the PartitionMessage

2017-09-22 Thread Eric Shu
Diff: https://reviews.apache.org/r/62506/diff/2/ Changes: https://reviews.apache.org/r/62506/diff/1-2/ Testing --- precheckin. Thanks, Eric Shu

Review Request 62506: GEODE-3679: Original client member Id in the transaction should be set in the PartitionMessage

2017-09-22 Thread Eric Shu
, Eric Shu

Re: [Discuss] Enable region set operations to start JTA transactions

2017-09-08 Thread Eric Shu
25, 2017 at 6:41 PM, Anilkumar Gingade <aging...@pivotal.io> > > wrote: > > > > > +1 Having a consistent behavior for set operation within the > transaction > > > (wherever its invoked). And having a gemfire property to allow old way. > > > >

Re: Review Request 61895: GEDOE-3516: TXManagerImpl.tryResume call may add itself again into the waiting thread queue

2017-09-06 Thread Eric Shu
: https://reviews.apache.org/r/61895/diff/2-3/ Testing --- precheckin. Thanks, Eric Shu

Re: Review Request 61895: GEDOE-3516: TXManagerImpl.tryResume call may add itself again into the waiting thread queue

2017-09-05 Thread Eric Shu
<https://reviews.apache.org/r/61895/#comment260751> The other two threads calls spyMgr.tryResume, and that method call will continue to park untill another thread calls suspend to unpark a waiting thread. Yes, it is guaranteed. - Eric Shu On Aug. 25, 2017, 4:53 p.m., Er

[Discuss] Enable region set operations to start JTA transactions

2017-08-25 Thread Eric Shu
Hi Team, Currently in GEODE JTA implementation, if GEODE transaction is not yet bootstrapped (there is no TXState created yet for the transaction), region set operations will not bootstrap the JTA transaction. In other word, if the first operation of the JTA transaction is region set operation,

Re: Review Request 61895: GEDOE-3516: TXManagerImpl.tryResume call may add itself again into the waiting thread queue

2017-08-25 Thread Eric Shu
/r/61895/diff/1-2/ Testing --- precheckin. Thanks, Eric Shu

Review Request 61895: GEDOE-3516: TXManagerImpl.tryResume call may add itself again into the waiting thread queue

2017-08-24 Thread Eric Shu
/TXManagerImpl.java a0a4d7c Diff: https://reviews.apache.org/r/61895/diff/1/ Testing --- precheckin. Thanks, Eric Shu

Re: Review Request 61852: GEODE-3507 actualRedundantCopies stat in org.apache.geode.internal.cache.PartitionedRegionStats can go negative

2017-08-23 Thread Eric Shu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61852/#review183627 --- Ship it! Ship It! - Eric Shu On Aug. 23, 2017, 4:55 p.m

Review Request 61676: Release the lock held when beforeCompletion failed with CommitConflictException

2017-08-15 Thread Eric Shu
/cache/TXState.java 55415e3 Diff: https://reviews.apache.org/r/61676/diff/1/ Testing --- Precheckin. Thanks, Eric Shu

Re: Review Request 61281: GEODE-3379 Geode transaction may commit on a secondary bucket after bucket rebalance

2017-08-02 Thread Eric Shu
/PRTransactionDUnitTest.java 6578baa Diff: https://reviews.apache.org/r/61281/diff/2/ Changes: https://reviews.apache.org/r/61281/diff/1-2/ Testing --- Ongoing precheckin. Thanks, Eric Shu

Review Request 61281: GEODE-3379 Geode transaction may commit on a secondary bucket after bucket rebalance

2017-08-01 Thread Eric Shu
--- Ongoing precheckin. Thanks, Eric Shu

Re: Review Request 61143: GEODE-3310 Set target node in TXStateProxy during TXFailover if necessary

2017-07-27 Thread Eric Shu
ontrol which node to failover to in the test -- you need to failover to another server which does not host the TXState. - Eric Shu On July 27, 2017, 5:32 p.m., Eric Shu wrote: > > --- > This is an automatically generated e-

Review Request 61143: GEODE-3310 Set target node in TXStateProxy during TXFailover if necessary

2017-07-26 Thread Eric Shu
/sockets/command/TXFailoverCommandTest.java PRE-CREATION Diff: https://reviews.apache.org/r/61143/diff/1/ Testing --- Ongoing precheckin. Thanks, Eric Shu

Re: Review Request 60874: GEODE-3204: txApplyInvalidate should not modify region entry with a removed token

2017-07-14 Thread Eric Shu
does not ack so that destroy message and invalidate message could be recieved out of order on the farside. - Eric --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60874/#r

Review Request 60874: GEODE-3204: txApplyInvalidate should not modify region entry with a removed token

2017-07-14 Thread Eric Shu
7f12eab geode-core/src/test/java/org/apache/geode/internal/cache/AbstractRegionMapTest.java 60b93a9 Diff: https://reviews.apache.org/r/60874/diff/1/ Testing --- precheckin. Thanks, Eric Shu

Review Request 60808: GEODE-3178 Fix a flaky JTA test

2017-07-12 Thread Eric Shu
: https://reviews.apache.org/r/60808/diff/1/ Testing --- UnitTest. Thanks, Eric Shu

Re: Review Request 60710: GEODE-3015: fix a flaky unit test

2017-07-07 Thread Eric Shu
/internal/cache/ClientServerTransactionDUnitTest.java ec4e179 Diff: https://reviews.apache.org/r/60710/diff/2/ Changes: https://reviews.apache.org/r/60710/diff/1-2/ Testing --- Unit test. Thanks, Eric Shu

Review Request 60710: GEODE-3015: fix a flaky unit test

2017-07-07 Thread Eric Shu
/60710/diff/1/ Testing --- Unit test. Thanks, Eric Shu

Re: Review Request 60513: GEODE-3147 Set TLCommBuffer threadLocal for threads executing TXSynchronization

2017-06-29 Thread Eric Shu
/java/org/apache/geode/internal/jta/ClientServerJTADUnitTest.java 51ef44a Diff: https://reviews.apache.org/r/60513/diff/3/ Changes: https://reviews.apache.org/r/60513/diff/2-3/ Testing --- precheckin. Thanks, Eric Shu

Re: Review Request 60513: GEODE-3147 Set TLCommBuffer threadLocal for threads executing TXSynchronization

2017-06-28 Thread Eric Shu
generated e-mail. To reply, visit: https://reviews.apache.org/r/60513/#review179198 ------- On June 29, 2017, 12:20 a.m., Eric Shu wrote: > > --- > This is an automat

Re: Review Request 60513: GEODE-3147 Set TLCommBuffer threadLocal for threads executing TXSynchronization

2017-06-28 Thread Eric Shu
This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60513/#review179156 ------- On June 29, 2017, 12:20 a.m., Eric Shu wrote: > >

Re: Review Request 60513: GEODE-3147 Set TLCommBuffer threadLocal for threads executing TXSynchronization

2017-06-28 Thread Eric Shu
/ClientServerJTADUnitTest.java 51ef44a Diff: https://reviews.apache.org/r/60513/diff/2/ Changes: https://reviews.apache.org/r/60513/diff/1-2/ Testing --- precheckin. Thanks, Eric Shu

Review Request 60513: GEODE-3147 Set TLCommBuffer threadLocal for threads executing TXSynchronization

2017-06-28 Thread Eric Shu
/command/TXSynchronizationCommand.java b1b0cfb geode-core/src/test/java/org/apache/geode/internal/jta/ClientServerJTADUnitTest.java 51ef44a Diff: https://reviews.apache.org/r/60513/diff/1/ Testing --- precheckin. Thanks, Eric Shu

Review Request 60447: GEODE-3132 EndBucketCreationMessage should not partitcipate in a transaction

2017-06-26 Thread Eric Shu
/EndBucketCreationMessage.java 355c3b5 Diff: https://reviews.apache.org/r/60447/diff/1/ Testing --- Precheckin. Thanks, Eric Shu

Review Request 60278: GEODE-3101 fix local locks not being released in client if the JTA transaction failed due to commit conflict on server in beforeCompletion()

2017-06-21 Thread Eric Shu
/ClientServerJTADUnitTest.java PRE-CREATION Diff: https://reviews.apache.org/r/60278/diff/1/ Testing --- precheckin ongoing. Thanks, Eric Shu

Re: Review Request 60051: GEODE-2301: Deprecate JTA TransactionManagerImpl

2017-06-14 Thread Eric Shu
://reviews.apache.org/r/60051/diff/1-2/ Testing --- Thanks, Eric Shu

Re: release 1.2

2017-06-14 Thread Eric Shu
I'd like to include GEODE-2301 in the release 1.2.0 (deprecate GEODE implementation of JTA transaction manager) -Eric On Wed, Jun 14, 2017 at 11:50 AM, Anthony Baker wrote: > Status update: > > - I think we have all changes needed in the release/1.2.0 branch [1]. > Please

Review Request 60051: GEODE-2301: Deprecate JTA TransactionManagerImpl

2017-06-13 Thread Eric Shu
/org/apache/geode/internal/jta/XidImpl.java 865f0aa Diff: https://reviews.apache.org/r/60051/diff/1/ Testing --- Thanks, Eric Shu

Re: Review Request 59849: add StringPrefixPartitionResolver

2017-06-06 Thread Eric Shu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59849/#review177074 --- Ship it! Ship It! - Eric Shu On June 6, 2017, 6:54 p.m

Re: Review Request 59849: add StringPrefixPartitionResolver

2017-06-06 Thread Eric Shu
/StringPrefixPartitionResolver.java Lines 60 (patched) <https://reviews.apache.org/r/59849/#comment250629> Does getName() needs the Delimiter being included? If it is used in the equals(), can we just check getDelimiter() in the equals method? - Eric Shu On June 6, 2017, 5:44 p.m.,

[jira] [Resolved] (GEODE-2892) Add sizeOnServer and isEmptyOnServer to Region

2017-06-01 Thread Eric Shu (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2892?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Eric Shu resolved GEODE-2892. - Resolution: Fixed > Add sizeOnServer and isEmptyOnServer to Reg

Re: Review Request 59564: GEODE-2892 implement sizeOnSever and isEmptyOnServer

2017-05-31 Thread Eric Shu
d1eddbb geode-core/src/test/java/org/apache/geode/security/SecurityTestUtils.java 2b26cdc Diff: https://reviews.apache.org/r/59564/diff/3/ Changes: https://reviews.apache.org/r/59564/diff/2-3/ Testing --- precheckin Thanks, Eric Shu

Re: Review Request 59564: GEODE-2892 implement sizeOnSever and isEmptyOnServer

2017-05-31 Thread Eric Shu
/ClientAuthenticationTestCase.java d1eddbb geode-core/src/test/java/org/apache/geode/security/SecurityTestUtils.java 2b26cdc Diff: https://reviews.apache.org/r/59564/diff/2/ Changes: https://reviews.apache.org/r/59564/diff/1-2/ Testing (updated) --- precheckin Thanks, Eric Shu

[jira] [Updated] (GEODE-3015) CI Failure: ClientServerTransactionDUnitTest.testTXStateCleanedUpIfJTABeforeCompletionFailedOnClient

2017-05-31 Thread Eric Shu (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-3015?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Eric Shu updated GEODE-3015: Labels: ci (was: ) > CI Fail

[jira] [Created] (GEODE-3015) CI Failure: ClientServerTransactionDUnitTest.testTXStateCleanedUpIfJTABeforeCompletionFailedOnClient

2017-05-31 Thread Eric Shu (JIRA)
Eric Shu created GEODE-3015: --- Summary: CI Failure: ClientServerTransactionDUnitTest.testTXStateCleanedUpIfJTABeforeCompletionFailedOnClient Key: GEODE-3015 URL: https://issues.apache.org/jira/browse/GEODE-3015

[jira] [Resolved] (GEODE-2939) Initiate bucket event tracker and get GII from different source could lead to bucket copies inconsistence

2017-05-25 Thread Eric Shu (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2939?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Eric Shu resolved GEODE-2939. - Resolution: Fixed Fix Version/s: 1.2.0 > Initiate bucket event tracker and get GII from differ

Re: Review Request 59404: GEODE-2939: make sure event tracker is initiated from the GII provider

2017-05-24 Thread Eric Shu
/BucketRegion.java Lines 293 (patched) <https://reviews.apache.org/r/59404/#comment249324> Only one thread will invoke this method from InitialImageOperation.getFromOne. - Eric Shu On May 25, 2017, 12:51 a.m., Eric Shu

Re: Review Request 59404: GEODE-2939: make sure event tracker is initiated from the GII provider

2017-05-24 Thread Eric Shu
This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59404/#review175956 --- On May 25, 2017, 12:51 a.m., Eric Shu wrote: > > -

Re: Review Request 59404: GEODE-2939: make sure event tracker is initiated from the GII provider

2017-05-24 Thread Eric Shu
fb5f0cf geode-core/src/test/java/org/apache/geode/internal/cache/EventTrackerDUnitTest.java 3faf41f Diff: https://reviews.apache.org/r/59404/diff/4/ Changes: https://reviews.apache.org/r/59404/diff/3-4/ Testing --- precheckin. Thanks, Eric Shu

Re: Review Request 59404: GEODE-2939: make sure event tracker is initiated from the GII provider

2017-05-23 Thread Eric Shu
geode-core/src/test/java/org/apache/geode/internal/cache/EventTrackerDUnitTest.java 3faf41f Diff: https://reviews.apache.org/r/59404/diff/3/ Changes: https://reviews.apache.org/r/59404/diff/2-3/ Testing --- precheckin. Thanks, Eric Shu

Re: Review Request 59404: GEODE-2939: make sure event tracker is initiated from the GII provider

2017-05-23 Thread Eric Shu
: https://reviews.apache.org/r/59404/diff/2/ Changes: https://reviews.apache.org/r/59404/diff/1-2/ Testing --- precheckin. Thanks, Eric Shu

Re: Review Request 59404: GEODE-2939: make sure event tracker is initiated from the GII provider

2017-05-23 Thread Eric Shu
ted e-mail. To reply, visit: https://reviews.apache.org/r/59404/#review175732 ------- On May 19, 2017, 3:38 p.m., Eric Shu wrote: > > --- > This is an automatically

Review Request 59455: GEODE-2916 Fix a flaky timing issue

2017-05-22 Thread Eric Shu
: https://reviews.apache.org/r/59455/diff/1/ Testing --- prechekin. Thanks, Eric Shu

[jira] [Assigned] (GEODE-2916) CI failure: CacheAdvisorDUnitTest.testNetLoadAdviceWithAttributesMutator

2017-05-22 Thread Eric Shu (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2916?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Eric Shu reassigned GEODE-2916: --- Assignee: Eric Shu > CI failure: CacheAdvisorDUnitTest.testNetLoadAdviceWithAttributesMuta

Review Request 59404: GEODE-2939: make sure event tracker is initiated from the GII provider

2017-05-19 Thread Eric Shu
/InitialImageOperation.java fb5f0cf geode-core/src/test/java/org/apache/geode/internal/cache/EventTrackerDUnitTest.java 3faf41f Diff: https://reviews.apache.org/r/59404/diff/1/ Testing --- precheckin. Thanks, Eric Shu

[jira] [Assigned] (GEODE-2939) Initiate bucket event tracker and get GII from different source could lead to bucket copies inconsistence

2017-05-18 Thread Eric Shu (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2939?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Eric Shu reassigned GEODE-2939: --- Assignee: Eric Shu > Initiate bucket event tracker and get GII from different source could l

[jira] [Created] (GEODE-2939) Initiate bucket event tracker and get GII from different source could lead to bucket copies inconsistence

2017-05-18 Thread Eric Shu (JIRA)
Eric Shu created GEODE-2939: --- Summary: Initiate bucket event tracker and get GII from different source could lead to bucket copies inconsistence Key: GEODE-2939 URL: https://issues.apache.org/jira/browse/GEODE-2939

[jira] [Resolved] (GEODE-2882) An IllegalStateException: attempting to hash null is thrown during transactional putAll when region is destroyed

2017-05-08 Thread Eric Shu (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2882?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Eric Shu resolved GEODE-2882. - Resolution: Fixed Fix Version/s: 1.2.0 > An IllegalStateException: attempting to hash n

Review Request 59032: GEODE-2882: Throw RegionDestroyedException instead of IllegalStateException when region is destroyed

2017-05-05 Thread Eric Shu
/PartitionedRegion.java da80fa6 geode-core/src/main/java/org/apache/geode/internal/cache/PartitionedRegionHelper.java 08dac6e Diff: https://reviews.apache.org/r/59032/diff/1/ Testing --- precheckin Thanks, Eric Shu

[jira] [Assigned] (GEODE-2882) An IllegalStateException: attempting to hash null is thrown during transactional putAll when region is destroyed

2017-05-05 Thread Eric Shu (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2882?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Eric Shu reassigned GEODE-2882: --- Assignee: Eric Shu > An IllegalStateException: attempting to hash null is thrown dur

[jira] [Created] (GEODE-2882) An IllegalStateException: attempting to hash null is thrown during transactional putAll when region is destroyed

2017-05-05 Thread Eric Shu (JIRA)
Eric Shu created GEODE-2882: --- Summary: An IllegalStateException: attempting to hash null is thrown during transactional putAll when region is destroyed Key: GEODE-2882 URL: https://issues.apache.org/jira/browse/GEODE

Re: Review Request 58966: remove meaningless tx warning message

2017-05-03 Thread Eric Shu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58966/#review173769 --- Ship it! Ship It! - Eric Shu On May 3, 2017, 5:14 p.m

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

2017-05-03 Thread Eric Shu (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Eric Shu resolved GEODE-2847. - Resolution: Fixed Fix Version/s: 1.2.0 > Retried bulk operation from client could not get appl

Re: Review Request 58853: GEODE-2847: Get correct version tags for retried bulk operations

2017-05-01 Thread Eric Shu
o longer using getEventTracker() call in the code so that the refactoring will be easier to do later to set the eventTracker field to be final. - Eric Shu On May 2, 2017, 1 a.m., Eric Shu wrote: > > --- > This is an automatically gen

Re: Review Request 58853: GEODE-2847: Get correct version tags for retried bulk operations

2017-05-01 Thread Eric Shu
Diff: https://reviews.apache.org/r/58853/diff/2/ Changes: https://reviews.apache.org/r/58853/diff/1-2/ Testing --- precheckin. Thanks, Eric Shu

Review Request 58853: GEODE-2847: Get correct version tags for retried bulk operations

2017-04-28 Thread Eric Shu
-CREATION Diff: https://reviews.apache.org/r/58853/diff/1/ Testing --- precheckin. Thanks, Eric Shu

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

2017-04-28 Thread Eric Shu (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Eric Shu reassigned GEODE-2847: --- Assignee: Eric Shu > Retried bulk operation from client could not get applied version tag f

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

2017-04-28 Thread Eric Shu (JIRA)
Eric Shu created GEODE-2847: --- Summary: Retried bulk operation from client could not get applied version tag from the server and cause data inconsistency Key: GEODE-2847 URL: https://issues.apache.org/jira/browse/GEODE

[jira] [Resolved] (GEODE-2785) CI failure: PartitionListenerDUnitTest.testAfterSecondaryIsCalledAfterLosingPrimary

2017-04-24 Thread Eric Shu (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2785?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Eric Shu resolved GEODE-2785. - Resolution: Fixed Fix Version/s: 1.2.0 > CI fail

[jira] [Assigned] (GEODE-2785) CI failure: PartitionListenerDUnitTest.testAfterSecondaryIsCalledAfterLosingPrimary

2017-04-24 Thread Eric Shu (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2785?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Eric Shu reassigned GEODE-2785: --- Assignee: Eric Shu > CI fail

Re: Review Request 58628: GEODE-2785: Fix a test issue with additional afterSecondary invocations

2017-04-24 Thread Eric Shu
) - geode-core/src/test/java/org/apache/geode/internal/cache/PartitionListenerDUnitTest.java 7fd470f Diff: https://reviews.apache.org/r/58628/diff/4/ Changes: https://reviews.apache.org/r/58628/diff/3-4/ Testing --- precheckin Thanks, Eric Shu

Re: Review Request 58628: GEODE-2785: Fix a test issue with additional afterSecondary invocations

2017-04-24 Thread Eric Shu
nk we may need to investage further. - Eric --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58628/#review172862 -------

Re: Review Request 58628: GEODE-2785: Fix a test issue with additional afterSecondary invocations

2017-04-21 Thread Eric Shu
) - geode-core/src/test/java/org/apache/geode/internal/cache/PartitionListenerDUnitTest.java 7fd470f Diff: https://reviews.apache.org/r/58628/diff/3/ Changes: https://reviews.apache.org/r/58628/diff/2-3/ Testing --- precheckin Thanks, Eric Shu

Re: Review Request 58628: GEODE-2785: Fix a test issue with additional afterSecondary invocations

2017-04-21 Thread Eric Shu
) - geode-core/src/test/java/org/apache/geode/internal/cache/PartitionListenerDUnitTest.java 9c38948 Diff: https://reviews.apache.org/r/58628/diff/2/ Changes: https://reviews.apache.org/r/58628/diff/1-2/ Testing --- precheckin Thanks, Eric Shu

  1   2   >