Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id D2DD5200D0F for ; Fri, 29 Sep 2017 17:01:06 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id D17A21609C5; Fri, 29 Sep 2017 15:01:06 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id 231CD1609D1 for ; Fri, 29 Sep 2017 17:01:05 +0200 (CEST) Received: (qmail 50556 invoked by uid 500); 29 Sep 2017 15:01:04 -0000 Mailing-List: contact issues-help@geode.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@geode.apache.org Delivered-To: mailing list issues@geode.apache.org Received: (qmail 50544 invoked by uid 99); 29 Sep 2017 15:01:04 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd1-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 29 Sep 2017 15:01:04 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd1-us-west.apache.org (ASF Mail Server at spamd1-us-west.apache.org) with ESMTP id F16DEC9C60 for ; Fri, 29 Sep 2017 15:01:03 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd1-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: -99.202 X-Spam-Level: X-Spam-Status: No, score=-99.202 tagged_above=-999 required=6.31 tests=[KAM_ASCII_DIVIDERS=0.8, RP_MATCHES_RCVD=-0.001, SPF_PASS=-0.001, USER_IN_WHITELIST=-100] autolearn=disabled Received: from mx1-lw-us.apache.org ([10.40.0.8]) by localhost (spamd1-us-west.apache.org [10.40.0.7]) (amavisd-new, port 10024) with ESMTP id uw22CjAvFfbg for ; Fri, 29 Sep 2017 15:01:03 +0000 (UTC) Received: from mailrelay1-us-west.apache.org (mailrelay1-us-west.apache.org [209.188.14.139]) by mx1-lw-us.apache.org (ASF Mail Server at mx1-lw-us.apache.org) with ESMTP id 072925FD33 for ; Fri, 29 Sep 2017 15:01:03 +0000 (UTC) Received: from jira-lw-us.apache.org (unknown [207.244.88.139]) by mailrelay1-us-west.apache.org (ASF Mail Server at mailrelay1-us-west.apache.org) with ESMTP id 4ABE4E0EEE for ; Fri, 29 Sep 2017 15:01:02 +0000 (UTC) Received: from jira-lw-us.apache.org (localhost [127.0.0.1]) by jira-lw-us.apache.org (ASF Mail Server at jira-lw-us.apache.org) with ESMTP id A4404242BE for ; Fri, 29 Sep 2017 15:01:01 +0000 (UTC) Date: Fri, 29 Sep 2017 15:01:01 +0000 (UTC) From: "Eric Shu (JIRA)" To: issues@geode.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Commented] (GEODE-3679) Server node should forward client member id to other partition nodes even if it already has a TXState MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 archived-at: Fri, 29 Sep 2017 15:01:07 -0000 [ https://issues.apache.org/jira/browse/GEODE-3679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16185942#comment-16185942 ] Eric Shu commented on GEODE-3679: --------------------------------- The fix reveals another issue in region.size() call in transaction. Will provide a fix of this as well. When a server starts transaction with PeerTXStateStub, (TXState resides on another server), it will send partitioned region.entryCount to the server hosting the tx. The hosting server gets size info of all the buckets from itself and all other nodes. Currently when the first server receive the request for size, it will go through the code path and sending the request back to the hosting server to get the size. This leads to the actual size information from the buckets it hosts to be lost. > Server node should forward client member id to other partition nodes even if it already has a TXState > ----------------------------------------------------------------------------------------------------- > > Key: GEODE-3679 > URL: https://issues.apache.org/jira/browse/GEODE-3679 > Project: Geode > Issue Type: Bug > Components: transactions > Affects Versions: 1.1.0, 1.2.0, 1.3.0 > Reporter: Eric Shu > Assignee: Eric Shu > > If the server starts the transaction from client with TXId(clientMember, uniqueId) and bootstrap a TXState as its realDeal, it should still forward the tx originating member to other nodes in a PartitionMessage as long as these msg can not start a new transaction ie FetchKeysMessage. > Otherwise, the receiving side will construct a transaction with TXId using the server memberId. There could be a case that the server did initiate a real transaction using the same uniqueId. This will cause problem, and other partition node may not process these message as the transaction may be already finished. -- This message was sent by Atlassian JIRA (v6.4.14#64029)