You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@calcite.apache.org by jh...@apache.org on 2016/02/25 21:35:11 UTC
calcite git commit: Further to [CALCITE-604], add "synchronized"
Repository: calcite
Updated Branches:
refs/heads/master d14040c52 -> b916a65b9
Further to [CALCITE-604], add "synchronized"
Project: http://git-wip-us.apache.org/repos/asf/calcite/repo
Commit: http://git-wip-us.apache.org/repos/asf/calcite/commit/b916a65b
Tree: http://git-wip-us.apache.org/repos/asf/calcite/tree/b916a65b
Diff: http://git-wip-us.apache.org/repos/asf/calcite/diff/b916a65b
Branch: refs/heads/master
Commit: b916a65b9615b8fff377f6b05991f445d43edf6c
Parents: d14040c
Author: Julian Hyde <jh...@apache.org>
Authored: Thu Feb 25 11:37:01 2016 -0800
Committer: Julian Hyde <jh...@apache.org>
Committed: Thu Feb 25 11:37:01 2016 -0800
----------------------------------------------------------------------
.../apache/calcite/rel/metadata/JaninoRelMetadataProvider.java | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/calcite/blob/b916a65b/core/src/main/java/org/apache/calcite/rel/metadata/JaninoRelMetadataProvider.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/calcite/rel/metadata/JaninoRelMetadataProvider.java b/core/src/main/java/org/apache/calcite/rel/metadata/JaninoRelMetadataProvider.java
index c861106..ac7f4d1 100644
--- a/core/src/main/java/org/apache/calcite/rel/metadata/JaninoRelMetadataProvider.java
+++ b/core/src/main/java/org/apache/calcite/rel/metadata/JaninoRelMetadataProvider.java
@@ -474,8 +474,10 @@ public class JaninoRelMetadataProvider implements RelMetadataProvider {
list.add(s);
}
}
- if (ALL_RELS.addAll(list)) {
- HANDLERS.invalidateAll();
+ synchronized (this) {
+ if (ALL_RELS.addAll(list)) {
+ HANDLERS.invalidateAll();
+ }
}
}