asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Blow (Code Review)" <do-not-re...@asterixdb.incubator.apache.org>
Subject Change in hyracks[master]: Fix Double-Checked Locking (Coverity)
Date Thu, 25 Feb 2016 14:41:13 GMT
Michael Blow has uploaded a new change for review.

  https://asterix-gerrit.ics.uci.edu/667

Change subject: Fix Double-Checked Locking (Coverity)
......................................................................

Fix Double-Checked Locking (Coverity)

Fix CIDs 68208,68209,68210 - double-checked locking requires volatile

Change-Id: I3383035b5e8e0aa3cfd54f803f12d56090f006cf
---
M hyracks/hyracks-storage-am-common/src/main/java/org/apache/hyracks/storage/am/common/dataflow/IndexDataflowHelper.java
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/hyracks refs/changes/67/667/1

diff --git a/hyracks/hyracks-storage-am-common/src/main/java/org/apache/hyracks/storage/am/common/dataflow/IndexDataflowHelper.java
b/hyracks/hyracks-storage-am-common/src/main/java/org/apache/hyracks/storage/am/common/dataflow/IndexDataflowHelper.java
index 7110e4b..57a8550 100644
--- a/hyracks/hyracks-storage-am-common/src/main/java/org/apache/hyracks/storage/am/common/dataflow/IndexDataflowHelper.java
+++ b/hyracks/hyracks-storage-am-common/src/main/java/org/apache/hyracks/storage/am/common/dataflow/IndexDataflowHelper.java
@@ -43,7 +43,7 @@
     protected final FileReference file;
     protected final int partition;
     protected final boolean durable;
-    protected IIndex index;
+    protected volatile IIndex index;
     protected final String resourcePath;
     protected final int resourcePartition;
 

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/667
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3383035b5e8e0aa3cfd54f803f12d56090f006cf
Gerrit-PatchSet: 1
Gerrit-Project: hyracks
Gerrit-Branch: master
Gerrit-Owner: Michael Blow <michael.blow@couchbase.com>

Mime
View raw message