abdullah alamoudi has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/2378
Change subject: [NO ISSUE] Only close the index in the close of the Upsert
Operator
......................................................................
[NO ISSUE] Only close the index in the close of the Upsert Operator
- user model changes: no
- storage format changes: no
- interface changes: no
details:
- Previously, a failure in the opening of the upsert operator will cause
the index to be closed twice.
- After this change, we only close the index in the close call of the
operator.
Change-Id: I1d76de2781362ef5c80aa926e1112e7872049c71
---
M
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/operators/LSMPrimaryUpsertOperatorNodePushable.java
1 file changed, 2 insertions(+), 4 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/78/2378/1
diff --git
a/asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/operators/LSMPrimaryUpsertOperatorNodePushable.java
b/asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/operators/LSMPrimaryUpsertOperatorNodePushable.java
index 21259cc..91bb9c8 100644
---
a/asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/operators/LSMPrimaryUpsertOperatorNodePushable.java
+++
b/asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/operators/LSMPrimaryUpsertOperatorNodePushable.java
@@ -241,10 +241,8 @@
appender.write(writer, true);
}
};
-
- } catch (Exception e) {
- indexHelper.close();
- throw new HyracksDataException(e);
+ } catch (Throwable e) { // NOSONAR: Re-thrown
+ throw HyracksDataException.create(e);
}
}
--
To view, visit https://asterix-gerrit.ics.uci.edu/2378
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I1d76de2781362ef5c80aa926e1112e7872049c71
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: abdullah alamoudi <[email protected]>
