Repository: calcite Updated Branches: refs/heads/master 1306040bd -> 3a4fba828
Removed unnecessary try/final block in RefCountPolicy (geode tests) Close apache/calcite#870 Project: http://git-wip-us.apache.org/repos/asf/calcite/repo Commit: http://git-wip-us.apache.org/repos/asf/calcite/commit/3a4fba82 Tree: http://git-wip-us.apache.org/repos/asf/calcite/tree/3a4fba82 Diff: http://git-wip-us.apache.org/repos/asf/calcite/diff/3a4fba82 Branch: refs/heads/master Commit: 3a4fba828f3d9e48749fadd22f134891612b7072 Parents: 1306040 Author: Andrei Sereda <[email protected]> Authored: Wed Sep 26 18:18:12 2018 -0400 Committer: Andrei Sereda <[email protected]> Committed: Wed Sep 26 18:47:33 2018 -0400 ---------------------------------------------------------------------- .../calcite/adapter/geode/rel/GeodeEmbeddedPolicy.java | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/calcite/blob/3a4fba82/geode/src/test/java/org/apache/calcite/adapter/geode/rel/GeodeEmbeddedPolicy.java ---------------------------------------------------------------------- diff --git a/geode/src/test/java/org/apache/calcite/adapter/geode/rel/GeodeEmbeddedPolicy.java b/geode/src/test/java/org/apache/calcite/adapter/geode/rel/GeodeEmbeddedPolicy.java index b1045b3..26c036f 100644 --- a/geode/src/test/java/org/apache/calcite/adapter/geode/rel/GeodeEmbeddedPolicy.java +++ b/geode/src/test/java/org/apache/calcite/adapter/geode/rel/GeodeEmbeddedPolicy.java @@ -132,13 +132,9 @@ public class GeodeEmbeddedPolicy extends ExternalResource { } @Override public synchronized void before() { - try { - if (refCount.get() == 0) { - // initialize only once - policy.before(); - } - } finally { - refCount.incrementAndGet(); + if (refCount.getAndIncrement() == 0) { + // initialize only once + policy.before(); } }
