Ian Maxon has submitted this change and it was merged.

Change subject: [ASTERIXDB-1706][API] Additional logging for debug
......................................................................


[ASTERIXDB-1706][API] Additional logging for debug

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

Details:
This is just another log, similar to the response size one
already merged, to try to nail down the true source of this
issue. The previous logs point to the issue being somewhere
besides result distribution.

Change-Id: Ie377e531688ecf715ef822ed0fb027ee124fc976
Reviewed-on: https://asterix-gerrit.ics.uci.edu/1929
Sonar-Qube: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
Integration-Tests: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
Tested-by: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
Reviewed-by: Murtadha Hubail <mhub...@apache.org>
---
M 
hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/ChunkedResponse.java
1 file changed, 5 insertions(+), 0 deletions(-)

Approvals:
  Murtadha Hubail: Looks good to me, approved
  Anon. E. Moose #1000171: 
  Jenkins: Verified; No violations found; Verified

Objections:
  Jenkins: 



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 1c30991..64be051 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
@@ -21,6 +21,8 @@
 import java.io.IOException;
 import java.io.OutputStream;
 import java.io.PrintWriter;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 import org.apache.hyracks.http.api.IServletResponse;
 
@@ -58,6 +60,8 @@
  * 4. larger than chunkSize, no error. -> header, data, empty response
  */
 public class ChunkedResponse implements IServletResponse {
+
+    private static final Logger LOGGER = 
Logger.getLogger(ChunkedResponse.class.getName());
     private final ChannelHandlerContext ctx;
     private final ChunkedNettyOutputStream outputStream;
     private final PrintWriter writer;
@@ -109,6 +113,7 @@
         } else {
             // There was an error
             if (headerSent) {
+                LOGGER.log(Level.WARNING,"Error after header write of chunked 
response");
                 if (error != null) {
                     error.release();
                 }

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie377e531688ecf715ef822ed0fb027ee124fc976
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon <ima...@apache.org>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Ian Maxon <ima...@apache.org>
Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mb...@apache.org>
Gerrit-Reviewer: Murtadha Hubail <mhub...@apache.org>
Gerrit-Reviewer: abdullah alamoudi <bamou...@gmail.com>

Reply via email to