Mailing-List: contact dev-help@geode.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@geode.apache.org
Date: Thu, 1 Jun 2017 20:53:04 +0000 (UTC)
From: "Jason Huynh (JIRA)" <jira@apache.org>
To: dev@geode.apache.org
Message-ID: <JIRA.13076650.1496350329000.345028.1496350384279@Atlassian.JIRA>
In-Reply-To: <JIRA.13076650.1496350329000@Atlassian.JIRA>
References: <JIRA.13076650.1496350329000@Atlassian.JIRA> <JIRA.13076650.1496350329169@jira-lw-us.apache.org>
Subject: [jira] [Assigned] (GEODE-3021) OQL Index isIndexedPdxKeys potential
 concurrency issue
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
archived-at: Thu, 01 Jun 2017 20:53:08 -0000


     [ https://issues.apache.org/jira/browse/GEODE-3021?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jason Huynh reassigned GEODE-3021:
----------------------------------

    Assignee: Jason Huynh

> OQL Index isIndexedPdxKeys potential concurrency issue
> ------------------------------------------------------
>
>                 Key: GEODE-3021
>                 URL: https://issues.apache.org/jira/browse/GEODE-3021
>             Project: Geode
>          Issue Type: Bug
>          Components: querying
>            Reporter: Jason Huynh
>            Assignee: Jason Huynh
>
> The boolean isIndexedPdxKeys is set in a synchronized method.  However multiple threads could be making a call to this method, the last caller would set the flag based on it's key, either true or false, but the previous callers could have meant to set it to the opposite.
> This method sets isIndexedPdxKeysFlagSet afterwards and hopes that callers to this method check this flag first.  However, again, multiple threads could have checked this flag, seen it as false, and then all attempted to call the synchronized method.  
> The solution is to probably check the isIndexedPdxKeysFlagSet flag in the method also.  That way, if another thread had already completed setPdxStringFlag(), the next call would be a no-op.
> The current issue will lead to possible class cast exceptions from String to PdxString or vice versa


--
This message was sent by Atlassian JIRA
(v6.3.15#6346)