>From Michael Blow <[email protected]>:
Michael Blow has uploaded this change for review. (
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17624 )
Change subject: [NO ISSUE][HYR][HTTP] Ensure OutputStream is closed in case of
writer.close() failure
......................................................................
[NO ISSUE][HYR][HTTP] Ensure OutputStream is closed in case of writer.close()
failure
Change-Id: Ibe5836eb61b3f01b8d95820f20c55269e98a3118
---
M
hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/ChunkedResponse.java
1 file changed, 15 insertions(+), 4 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/24/17624/1
diff --git
a/hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/ChunkedResponse.java
b/hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/ChunkedResponse.java
index e00c519..ee6d9be 100644
---
a/hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/ChunkedResponse.java
+++
b/hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/ChunkedResponse.java
@@ -114,10 +114,12 @@
@Override
public void close() throws IOException {
- if (writer != null) {
- writer.close();
- } else {
- outputStream.close();
+ try {
+ if (writer != null) {
+ writer.close();
+ }
+ } finally {
+ outputStream.close();
}
if (errorBuf == null && response.status() == HttpResponseStatus.OK) {
if (!done) {
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17624
To unsubscribe, or for help writing mail filters, visit
https://asterix-gerrit.ics.uci.edu/settings
Gerrit-Project: asterixdb
Gerrit-Branch: neo
Gerrit-Change-Id: Ibe5836eb61b3f01b8d95820f20c55269e98a3118
Gerrit-Change-Number: 17624
Gerrit-PatchSet: 1
Gerrit-Owner: Michael Blow <[email protected]>
Gerrit-MessageType: newchange
