Change in asterixdb[master]: [ASTERIXDB-1706][API] Additional logging for debug

2017-08-09 Thread Ian Maxon (Code Review) 
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 
Integration-Tests: Jenkins 
Tested-by: Jenkins 
Reviewed-by: Murtadha Hubail 
---
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 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-Reviewer: abdullah alamoudi

Change in asterixdb[master]: [ASTERIXDB-1706][API] Additional logging for debug

2017-08-09 Thread Murtadha Hubail (Code Review) 
Murtadha Hubail has posted comments on this change.

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


Patch Set 1: Code-Review+2

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie377e531688ecf715ef822ed0fb027ee124fc976
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-1706][API] Additional logging for debug

2017-08-09 Thread Anon. E. Moose (Code Review) 
Anon. E. Moose #1000171 has posted comments on this change.

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


Patch Set 1: Contrib+1

Analytics Compatibility Tests Successful Build 2424
: SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie377e531688ecf715ef822ed0fb027ee124fc976
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-1706][API] Additional logging for debug

2017-08-09 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

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


Patch Set 1: Contrib-1

BAD Compatibility Tests Failed

https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/1606/ : FAILURE

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie377e531688ecf715ef822ed0fb027ee124fc976
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-1706][API] Additional logging for debug

2017-08-09 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

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


Patch Set 1:

BAD Compatibility Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/1606/

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie377e531688ecf715ef822ed0fb027ee124fc976
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-1706][API] Additional logging for debug

2017-08-09 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

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


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/1253/
 (11/11)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie377e531688ecf715ef822ed0fb027ee124fc976
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-1706][API] Additional logging for debug

2017-08-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

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


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/1300/ 
(11/11)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie377e531688ecf715ef822ed0fb027ee124fc976
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-1706][API] Additional logging for debug

2017-08-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

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


Patch Set 1: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/3646/ 
: SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie377e531688ecf715ef822ed0fb027ee124fc976
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-1706][API] Additional logging for debug

2017-08-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

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


Patch Set 1:

Integration Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/3646/

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie377e531688ecf715ef822ed0fb027ee124fc976
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-1706][API] Additional logging for debug

2017-08-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

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


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/1295/ 
(11/11)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie377e531688ecf715ef822ed0fb027ee124fc976
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-1706][API] Additional logging for debug

2017-08-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

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


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/558/
 (10/11)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie377e531688ecf715ef822ed0fb027ee124fc976
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-1706][API] Additional logging for debug

2017-08-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

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


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/1245/
 (9/11)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie377e531688ecf715ef822ed0fb027ee124fc976
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-1706][API] Additional logging for debug

2017-08-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

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


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app/1468/ (8/11)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie377e531688ecf715ef822ed0fb027ee124fc976
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-1706][API] Additional logging for debug

2017-08-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

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


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/6127/ (7/11)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie377e531688ecf715ef822ed0fb027ee124fc976
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-1706][API] Additional logging for debug

2017-08-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

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


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/4646/ (2/11)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie377e531688ecf715ef822ed0fb027ee124fc976
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-1706][API] Additional logging for debug

2017-08-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

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


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/565/ (5/11)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie377e531688ecf715ef822ed0fb027ee124fc976
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-1706][API] Additional logging for debug

2017-08-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

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


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-rebalance-cancellation/385/
 (3/11)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie377e531688ecf715ef822ed0fb027ee124fc976
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-1706][API] Additional logging for debug

2017-08-08 Thread Ian Maxon (Code Review) 
Ian Maxon has uploaded a new change for review.

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

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
---
M 
hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/ChunkedResponse.java
1 file changed, 5 insertions(+), 0 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/29/1929/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 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: newchange
Gerrit-Change-Id: Ie377e531688ecf715ef822ed0fb027ee124fc976
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon

Change in asterixdb[master]: [ASTERIXDB-1706][API] Additional logging for debug

2017-08-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

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


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/570/ 
(6/11)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie377e531688ecf715ef822ed0fb027ee124fc976
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-1706][API] Additional logging for debug

2017-08-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

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


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/903/ 
(4/11)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie377e531688ecf715ef822ed0fb027ee124fc976
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: [ASTERIXDB-1706][API] Additional logging for debug

2017-08-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

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


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/1183/ (1/11)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie377e531688ecf715ef822ed0fb027ee124fc976
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No