abdullah alamoudi has uploaded a new change for review.

  https://asterix-gerrit.ics.uci.edu/2403

Change subject: [NO ISSUE][OTH] Remove logs for legal destroy calls
......................................................................

[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
---
M 
hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/EnforcedIndexCursor.java
1 file changed, 0 insertions(+), 2 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/03/2403/1

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 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) {

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2403
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8fc5c4ad165e44d5259446aad8e64826a672fd77
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: abdullah alamoudi <bamou...@gmail.com>

Reply via email to