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 <[email protected]>
