You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@asterixdb.apache.org by "Murtadha Hubail (Code Review)" <de...@asterixdb.apache.org> on 2019/05/09 12:18:40 UTC

Change in asterixdb[master]: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized

Murtadha Hubail has uploaded this change for review. ( https://asterix-gerrit.ics.uci.edu/3389


Change subject: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized
......................................................................

[NO ISSUE][OTH] Ensure HttpServer Channel is Initialized

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

Details:
- Ensure the HttpServer channel is initialized when stopping
  the server to avoid NPE.

Change-Id: I5b7403e80f6118f99be46d166c6cfbee8d4305ac
---
M hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/HttpServer.java
1 file changed, 4 insertions(+), 2 deletions(-)



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

diff --git a/hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/HttpServer.java b/hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/HttpServer.java
index 152f6c7..d9902da 100644
--- a/hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/HttpServer.java
+++ b/hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/HttpServer.java
@@ -333,8 +333,10 @@
         } catch (Exception e) {
             LOGGER.log(Level.ERROR, "Error while shutting down http server executor", e);
         }
-        channel.close();
-        channel.closeFuture().sync();
+        if (channel != null) {
+            channel.close();
+            channel.closeFuture().sync();
+        }
     }
 
     public IServlet getServlet(FullHttpRequest request) {

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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I5b7403e80f6118f99be46d166c6cfbee8d4305ac
Gerrit-Change-Number: 3389
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail <mh...@apache.org>

Change in asterixdb[master]: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized

Posted by "Murtadha Hubail (Code Review)" <de...@asterixdb.apache.org>.
Murtadha Hubail has submitted this change and it was merged. ( https://asterix-gerrit.ics.uci.edu/3389 )

Change subject: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized
......................................................................

[NO ISSUE][OTH] Ensure HttpServer Channel is Initialized

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

Details:
- Ensure the HttpServer channel is initialized when stopping
  the server to avoid NPE.

Change-Id: I5b7403e80f6118f99be46d166c6cfbee8d4305ac
Reviewed-on: https://asterix-gerrit.ics.uci.edu/3389
Contrib: Jenkins <je...@fulliautomatix.ics.uci.edu>
Tested-by: Jenkins <je...@fulliautomatix.ics.uci.edu>
Integration-Tests: Jenkins <je...@fulliautomatix.ics.uci.edu>
Reviewed-by: Dmitry Lychagin <dm...@couchbase.com>
---
M hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/HttpServer.java
1 file changed, 4 insertions(+), 2 deletions(-)

Approvals:
  Jenkins: Verified; ; Verified
  Anon. E. Moose (1000171): 
  Dmitry Lychagin: Looks good to me, approved

Objections:
  Jenkins: Violations found



diff --git a/hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/HttpServer.java b/hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/HttpServer.java
index 152f6c7..d9902da 100644
--- a/hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/HttpServer.java
+++ b/hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/HttpServer.java
@@ -333,8 +333,10 @@
         } catch (Exception e) {
             LOGGER.log(Level.ERROR, "Error while shutting down http server executor", e);
         }
-        channel.close();
-        channel.closeFuture().sync();
+        if (channel != null) {
+            channel.close();
+            channel.closeFuture().sync();
+        }
     }
 
     public IServlet getServlet(FullHttpRequest request) {

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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I5b7403e80f6118f99be46d166c6cfbee8d4305ac
Gerrit-Change-Number: 3389
Gerrit-PatchSet: 2
Gerrit-Owner: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Ali Alsuliman <al...@gmail.com>
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Dmitry Lychagin <dm...@couchbase.com>
Gerrit-Reviewer: Hussain Towaileb <hu...@gmail.com>
Gerrit-Reviewer: Ian Maxon <im...@uci.edu>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mb...@apache.org>
Gerrit-Reviewer: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>

Change in asterixdb[master]: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized

Posted by "Jenkins (Code Review)" <de...@asterixdb.apache.org>.
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3389 )

Change subject: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/6123/ (10/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b7403e80f6118f99be46d166c6cfbee8d4305ac
Gerrit-Change-Number: 3389
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Comment-Date: Thu, 09 May 2019 12:21:36 +0000
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized

Posted by "Jenkins (Code Review)" <de...@asterixdb.apache.org>.
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3389 )

Change subject: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/9709/ (9/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b7403e80f6118f99be46d166c6cfbee8d4305ac
Gerrit-Change-Number: 3389
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Comment-Date: Thu, 09 May 2019 12:20:07 +0000
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized

Posted by "Jenkins (Code Review)" <de...@asterixdb.apache.org>.
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3389 )

Change subject: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-format/5729/ (4/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b7403e80f6118f99be46d166c6cfbee8d4305ac
Gerrit-Change-Number: 3389
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Comment-Date: Thu, 09 May 2019 12:18:49 +0000
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized

Posted by "Jenkins (Code Review)" <de...@asterixdb.apache.org>.
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3389 )

Change subject: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/6333/ (12/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b7403e80f6118f99be46d166c6cfbee8d4305ac
Gerrit-Change-Number: 3389
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Comment-Date: Thu, 09 May 2019 12:23:05 +0000
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized

Posted by "Jenkins (Code Review)" <de...@asterixdb.apache.org>.
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3389 )

Change subject: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ensure-ancestor/3763/ (14/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b7403e80f6118f99be46d166c6cfbee8d4305ac
Gerrit-Change-Number: 3389
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Comment-Date: Thu, 09 May 2019 12:26:15 +0000
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized

Posted by "Jenkins (Code Review)" <de...@asterixdb.apache.org>.
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3389 )

Change subject: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/5666/ (3/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b7403e80f6118f99be46d166c6cfbee8d4305ac
Gerrit-Change-Number: 3389
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Comment-Date: Thu, 09 May 2019 12:18:49 +0000
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized

Posted by "Jenkins (Code Review)" <de...@asterixdb.apache.org>.
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3389 )

Change subject: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized
......................................................................


Patch Set 1:

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


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b7403e80f6118f99be46d166c6cfbee8d4305ac
Gerrit-Change-Number: 3389
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Comment-Date: Thu, 09 May 2019 12:20:38 +0000
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized

Posted by "Jenkins (Code Review)" <de...@asterixdb.apache.org>.
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3389 )

Change subject: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-txnlog/929/ (1/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b7403e80f6118f99be46d166c6cfbee8d4305ac
Gerrit-Change-Number: 3389
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Comment-Date: Thu, 09 May 2019 12:18:48 +0000
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized

Posted by "Jenkins (Code Review)" <de...@asterixdb.apache.org>.
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3389 )

Change subject: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ssl-compression/532/ (5/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b7403e80f6118f99be46d166c6cfbee8d4305ac
Gerrit-Change-Number: 3389
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Comment-Date: Thu, 09 May 2019 12:18:50 +0000
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized

Posted by "Anon. E. Moose (Code Review)" <de...@asterixdb.apache.org>.
Anon. E. Moose (1000171) has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3389 )

Change subject: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized
......................................................................


Patch Set 1: Contrib+1

Analytics Compatibility Tests Successful
https://cbjenkins.page.link/LK2F6TQn4Qs8Sjiq7 : SUCCESS


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b7403e80f6118f99be46d166c6cfbee8d4305ac
Gerrit-Change-Number: 3389
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Comment-Date: Thu, 09 May 2019 15:48:49 +0000
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized

Posted by "Jenkins (Code Review)" <de...@asterixdb.apache.org>.
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3389 )

Change subject: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/5766/ (8/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b7403e80f6118f99be46d166c6cfbee8d4305ac
Gerrit-Change-Number: 3389
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Comment-Date: Thu, 09 May 2019 12:18:52 +0000
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized

Posted by "Jenkins (Code Review)" <de...@asterixdb.apache.org>.
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3389 )

Change subject: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-spidersilk-tests/692/ (6/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b7403e80f6118f99be46d166c6cfbee8d4305ac
Gerrit-Change-Number: 3389
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Comment-Date: Thu, 09 May 2019 12:18:50 +0000
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized

Posted by "Jenkins (Code Review)" <de...@asterixdb.apache.org>.
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3389 )

Change subject: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-stabilization-f69489-compat/1027/ (11/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b7403e80f6118f99be46d166c6cfbee8d4305ac
Gerrit-Change-Number: 3389
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Comment-Date: Thu, 09 May 2019 12:22:26 +0000
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized

Posted by "Jenkins (Code Review)" <de...@asterixdb.apache.org>.
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3389 )

Change subject: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized
......................................................................


Patch Set 1: Integration-Tests+1

Integration Tests Successful

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


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b7403e80f6118f99be46d166c6cfbee8d4305ac
Gerrit-Change-Number: 3389
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Comment-Date: Thu, 09 May 2019 13:59:21 +0000
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized

Posted by "Anon. E. Moose (Code Review)" <de...@asterixdb.apache.org>.
Anon. E. Moose (1000171) has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3389 )

Change subject: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized
......................................................................


Patch Set 1:

Analytics Compatibility Compilation Successful
https://cbjenkins.page.link/PvvLAdNk7y8yumJ56 : SUCCESS


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b7403e80f6118f99be46d166c6cfbee8d4305ac
Gerrit-Change-Number: 3389
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Comment-Date: Thu, 09 May 2019 12:23:05 +0000
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized

Posted by "Jenkins (Code Review)" <de...@asterixdb.apache.org>.
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3389 )

Change subject: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized
......................................................................


Patch Set 1:

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


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b7403e80f6118f99be46d166c6cfbee8d4305ac
Gerrit-Change-Number: 3389
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Comment-Date: Thu, 09 May 2019 12:42:31 +0000
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized

Posted by "Jenkins (Code Review)" <de...@asterixdb.apache.org>.
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3389 )

Change subject: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized
......................................................................


Patch Set 1: Contrib+1

BAD Compatibility Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/4389/ : SUCCESS


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b7403e80f6118f99be46d166c6cfbee8d4305ac
Gerrit-Change-Number: 3389
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Comment-Date: Thu, 09 May 2019 12:35:09 +0000
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized

Posted by "Jenkins (Code Review)" <de...@asterixdb.apache.org>.
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3389 )

Change subject: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-openjdk11/1119/ (16/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b7403e80f6118f99be46d166c6cfbee8d4305ac
Gerrit-Change-Number: 3389
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Comment-Date: Thu, 09 May 2019 12:46:26 +0000
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized

Posted by "Dmitry Lychagin (Code Review)" <de...@asterixdb.apache.org>.
Dmitry Lychagin has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3389 )

Change subject: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized
......................................................................


Patch Set 1: Code-Review+2


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b7403e80f6118f99be46d166c6cfbee8d4305ac
Gerrit-Change-Number: 3389
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Ali Alsuliman <al...@gmail.com>
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Dmitry Lychagin <dm...@couchbase.com>
Gerrit-Reviewer: Hussain Towaileb <hu...@gmail.com>
Gerrit-Reviewer: Ian Maxon <im...@uci.edu>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mb...@apache.org>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-Comment-Date: Thu, 09 May 2019 17:18:21 +0000
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized

Posted by "Jenkins (Code Review)" <de...@asterixdb.apache.org>.
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3389 )

Change subject: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/11240/ (2/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b7403e80f6118f99be46d166c6cfbee8d4305ac
Gerrit-Change-Number: 3389
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Comment-Date: Thu, 09 May 2019 12:18:49 +0000
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized

Posted by "Jenkins (Code Review)" <de...@asterixdb.apache.org>.
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3389 )

Change subject: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized
......................................................................


Patch Set 1:

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


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b7403e80f6118f99be46d166c6cfbee8d4305ac
Gerrit-Change-Number: 3389
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Comment-Date: Thu, 09 May 2019 12:46:47 +0000
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized

Posted by "Jenkins (Code Review)" <de...@asterixdb.apache.org>.
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3389 )

Change subject: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/5750/ (7/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b7403e80f6118f99be46d166c6cfbee8d4305ac
Gerrit-Change-Number: 3389
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Comment-Date: Thu, 09 May 2019 12:18:51 +0000
Gerrit-HasComments: No

Change in asterixdb[master]: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized

Posted by "Jenkins (Code Review)" <de...@asterixdb.apache.org>.
Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/3389 )

Change subject: [NO ISSUE][OTH] Ensure HttpServer Channel is Initialized
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/5973/ (13/16)


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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b7403e80f6118f99be46d166c6cfbee8d4305ac
Gerrit-Change-Number: 3389
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Comment-Date: Thu, 09 May 2019 12:26:10 +0000
Gerrit-HasComments: No