Repository: hive Updated Branches: refs/heads/master db239d52d -> 247e884d0
HIVE-13163: ORC MemoryManager thread checks are fatal, should WARN (Gopal V, reviewed by Prasanth Jayachandran) Project: http://git-wip-us.apache.org/repos/asf/hive/repo Commit: http://git-wip-us.apache.org/repos/asf/hive/commit/a3737498 Tree: http://git-wip-us.apache.org/repos/asf/hive/tree/a3737498 Diff: http://git-wip-us.apache.org/repos/asf/hive/diff/a3737498 Branch: refs/heads/master Commit: a3737498379a67d76d37684bb083772200a4cf16 Parents: db239d5 Author: Gopal V <[email protected]> Authored: Wed Mar 2 22:45:17 2016 -0800 Committer: Gopal V <[email protected]> Committed: Wed Mar 2 22:45:17 2016 -0800 ---------------------------------------------------------------------- orc/src/java/org/apache/orc/impl/MemoryManager.java | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hive/blob/a3737498/orc/src/java/org/apache/orc/impl/MemoryManager.java ---------------------------------------------------------------------- diff --git a/orc/src/java/org/apache/orc/impl/MemoryManager.java b/orc/src/java/org/apache/orc/impl/MemoryManager.java index 2dbfba7..757c0b4 100644 --- a/orc/src/java/org/apache/orc/impl/MemoryManager.java +++ b/orc/src/java/org/apache/orc/impl/MemoryManager.java @@ -101,10 +101,10 @@ public class MemoryManager { * Light weight thread-safety check for multi-threaded access patterns */ private void checkOwner() { - Preconditions.checkArgument(ownerLock.isHeldByCurrentThread(), - "Owner thread expected %s, got %s", - ownerLock.getOwner(), - Thread.currentThread()); + if (!ownerLock.isHeldByCurrentThread()) { + LOG.warn("Owner thread expected {}, got {}", + ownerLock.getOwner(), Thread.currentThread()); + } } /**
