You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@asterixdb.apache.org by am...@apache.org on 2018/02/19 22:44:56 UTC

asterixdb git commit: [NO ISSUE][OTH] Remove logs for legal destroy calls

Repository: asterixdb
Updated Branches:
  refs/heads/master 18ef4b99a -> 15589b42e


[NO ISSUE][OTH] Remove logs for legal destroy calls

- user model changes: no
- storage format changes: no
- interface changes: no

details:
- Previously, we logged when a cursor was destroyed multiple times.
- Since this is a legal call, we don't log this anymore.

Change-Id: I8fc5c4ad165e44d5259446aad8e64826a672fd77
Reviewed-on: https://asterix-gerrit.ics.uci.edu/2403
Sonar-Qube: Jenkins <je...@fulliautomatix.ics.uci.edu>
Tested-by: Jenkins <je...@fulliautomatix.ics.uci.edu>
Contrib: Jenkins <je...@fulliautomatix.ics.uci.edu>
Reviewed-by: Murtadha Hubail <mh...@apache.org>
Integration-Tests: Jenkins <je...@fulliautomatix.ics.uci.edu>


Project: http://git-wip-us.apache.org/repos/asf/asterixdb/repo
Commit: http://git-wip-us.apache.org/repos/asf/asterixdb/commit/15589b42
Tree: http://git-wip-us.apache.org/repos/asf/asterixdb/tree/15589b42
Diff: http://git-wip-us.apache.org/repos/asf/asterixdb/diff/15589b42

Branch: refs/heads/master
Commit: 15589b42e590da67b9b5392541e58827ac1bb804
Parents: 18ef4b9
Author: Abdullah Alamoudi <ba...@gmail.com>
Authored: Mon Feb 19 11:01:24 2018 -0800
Committer: abdullah alamoudi <ba...@gmail.com>
Committed: Mon Feb 19 14:44:22 2018 -0800

----------------------------------------------------------------------
 .../org/apache/hyracks/storage/common/EnforcedIndexCursor.java     | 2 --
 1 file changed, 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/asterixdb/blob/15589b42/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/EnforcedIndexCursor.java
----------------------------------------------------------------------
diff --git a/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/EnforcedIndexCursor.java b/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/EnforcedIndexCursor.java
index 929faff..b7caf7f 100644
--- a/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/EnforcedIndexCursor.java
+++ b/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/EnforcedIndexCursor.java
@@ -85,8 +85,6 @@ public abstract class EnforcedIndexCursor implements IIndexCursor {
     public final void destroy() throws HyracksDataException {
         if (ENFORCE_OPEN_CLOSE_DESTROY) {
             if (state == State.DESTROYED) {
-                LOGGER.log(Level.WARN,
-                        "multiple cursor.destroy() call in " + Arrays.toString(new Throwable().getStackTrace()));
                 return;
             } else if (state != State.CLOSED) {
                 if (STORE_TRACES && openCallStack != null) {