Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Murtadha Hubail has submitted this change and it was merged. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. [NO ISSUE][API] Add NC Query Cancellation API - user model changes: no - storage format changes: no - interface changes: no Details: - Introduce NCQueryCancellationServlet that uses NC<->CC messaging to cancel queries. Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Reviewed-on: https://asterix-gerrit.ics.uci.edu/2103 Reviewed-by: Michael BlowSonar-Qube: Jenkins Tested-by: Jenkins Contrib: Jenkins Integration-Tests: Jenkins --- A asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/NCQueryCancellationServlet.java M asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryCancellationServlet.java 2 files changed, 77 insertions(+), 1 deletion(-) Approvals: Jenkins: Verified; No violations found; ; Verified Michael Blow: Looks good to me, approved Objections: Anon. E. Moose #1000171: diff --git a/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/NCQueryCancellationServlet.java b/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/NCQueryCancellationServlet.java new file mode 100644 index 000..637c492 --- /dev/null +++ b/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/NCQueryCancellationServlet.java @@ -0,0 +1,76 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreements. See the NOTICE file + * distributed with this work for additional information + * regarding copyright ownership. The ASF licenses this file + * to you under the Apache License, Version 2.0 (the + * "License"); you may not use this file except in compliance + * with the License. You may obtain a copy of the License at + * + *http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, + * software distributed under the License is distributed on an + * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY + * KIND, either express or implied. See the License for the + * specific language governing permissions and limitations + * under the License. + */ +package org.apache.asterix.api.http.server; + +import static org.apache.asterix.app.message.ExecuteStatementRequestMessage.DEFAULT_NC_TIMEOUT_MILLIS; + +import java.io.IOException; +import java.util.concurrent.ConcurrentMap; +import java.util.concurrent.TimeUnit; +import java.util.logging.Level; +import java.util.logging.Logger; + +import org.apache.asterix.app.message.CancelQueryRequest; +import org.apache.asterix.common.messaging.api.INCMessageBroker; +import org.apache.asterix.common.messaging.api.MessageFuture; +import org.apache.hyracks.api.application.INCServiceContext; +import org.apache.hyracks.http.api.IServletRequest; +import org.apache.hyracks.http.api.IServletResponse; + +import io.netty.handler.codec.http.HttpResponseStatus; + +/** + * The servlet provides a REST API on an NC for cancelling an on-going query. + */ +public class NCQueryCancellationServlet extends QueryCancellationServlet { +private static final Logger LOGGER = Logger.getLogger(NCQueryCancellationServlet.class.getName()); +private final INCServiceContext serviceCtx; +private final INCMessageBroker messageBroker; + +public NCQueryCancellationServlet(ConcurrentMap ctx, String... paths) { +super(ctx, paths); +this.serviceCtx = (INCServiceContext) ctx.get(ServletConstants.SERVICE_CONTEXT_ATTR); +messageBroker = (INCMessageBroker) serviceCtx.getMessageBroker(); +} + +@Override +protected void delete(IServletRequest request, IServletResponse response) throws IOException { +// gets the parameter client_context_id from the request. +String clientContextId = request.getParameter(CLIENT_CONTEXT_ID); +if (clientContextId == null) { +response.setStatus(HttpResponseStatus.BAD_REQUEST); +return; +} +final MessageFuture cancelQueryFuture = messageBroker.registerMessageFuture(); +try { +CancelQueryRequest cancelQueryMessage = +new CancelQueryRequest(serviceCtx.getNodeId(), cancelQueryFuture.getFutureId(), clientContextId); +messageBroker.sendMessageToCC(cancelQueryMessage); +cancelQueryFuture.get(DEFAULT_NC_TIMEOUT_MILLIS, TimeUnit.MILLISECONDS); +response.setStatus(HttpResponseStatus.OK); +} catch (Exception e) { +if (LOGGER.isLoggable(Level.SEVERE)) { +LOGGER.log(Level.SEVERE, "Unexpected exception while canceling query", e); +} +
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Anon. E. Moose #1000171 has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 3: Contrib-1 Analytics Compatibility Tests Failed Build 2831 : UNSTABLE -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 3: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/4373/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 3: Contrib+1 BAD Compatibility Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/2010/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 3: BAD Compatibility Tests Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/2010/ -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 3: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/4373/ -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/1323/ (10/10) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/6867/ (7/10) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/1930/ (5/10) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/1651/ (9/10) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Michael Blow has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 3: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/2023/ (3/10) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app/2210/ (4/10) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/1324/ (1/10) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/1314/ (2/10) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/5371/ (8/10) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/1989/ (6/10) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Michael Blow has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 2: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Anon. E. Moose #1000171 has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 2: Contrib-1 Analytics Compatibility Tests Failed Build 2824 : UNSTABLE -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 2: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/4363/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 2: Contrib+1 BAD Compatibility Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/2003/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 2: BAD Compatibility Tests Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/2003/ -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 2: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/4363/ -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app/2199/ (9/10) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/2013/ (10/10) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/1920/ (7/10) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/1313/ (3/5) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/5361/ (8/10) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/1314/ (4/5) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/1979/ (5/5) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/1641/ (1/5) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/6857/ (2/5) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/1304/ (6/8) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Anon. E. Moose #1000171 has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 1: Contrib-1 Analytics Compatibility Tests Failed Build 2823 : UNSTABLE -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2103 to look at the new patch set (#2). Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. [NO ISSUE][API] Add NC Query Cancellation API - user model changes: no - storage format changes: no - interface changes: no Details: - Introduce NCQueryCancellationServlet that uses NC<->CC messaging to cancel queries. Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 --- A asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/NCQueryCancellationServlet.java M asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryCancellationServlet.java 2 files changed, 77 insertions(+), 1 deletion(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/03/2103/2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 1: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/4362/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 1: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/4362/ -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/1303/ (7/9) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/1312/ (9/10) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/1313/ (4/5) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app/2198/ (2/4) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/1919/ (5/7) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/2012/ (10/10) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/1640/ (6/7) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/5360/ (8/10) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/6856/ (3/5) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Jenkins has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/1978/ (1/3) -- To view, visit https://asterix-gerrit.ics.uci.edu/2103 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][API] Add NC Query Cancellation API
Murtadha Hubail has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2103 Change subject: [NO ISSUE][API] Add NC Query Cancellation API .. [NO ISSUE][API] Add NC Query Cancellation API - user model changes: no - storage format changes: no - interface changes: no Details: - Introduce NCQueryCancellationServlet that uses NC<->CC messaging to cancel queries. Change-Id: I75d326b6987ed87ccc4b58c171bd45cab33a4009 --- A asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/NCQueryCancellationServlet.java M asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryCancellationServlet.java 2 files changed, 73 insertions(+), 1 deletion(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/03/2103/1 diff --git a/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/NCQueryCancellationServlet.java b/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/NCQueryCancellationServlet.java new file mode 100644 index 000..191ef22 --- /dev/null +++ b/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/NCQueryCancellationServlet.java @@ -0,0 +1,72 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreements. See the NOTICE file + * distributed with this work for additional information + * regarding copyright ownership. The ASF licenses this file + * to you under the Apache License, Version 2.0 (the + * "License"); you may not use this file except in compliance + * with the License. You may obtain a copy of the License at + * + *http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, + * software distributed under the License is distributed on an + * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY + * KIND, either express or implied. See the License for the + * specific language governing permissions and limitations + * under the License. + */ +package org.apache.asterix.api.http.server; + +import java.io.IOException; +import java.util.concurrent.ConcurrentMap; +import java.util.logging.Level; +import java.util.logging.Logger; + +import org.apache.asterix.app.message.CancelQueryRequest; +import org.apache.asterix.common.messaging.api.INCMessageBroker; +import org.apache.asterix.common.messaging.api.MessageFuture; +import org.apache.hyracks.api.application.INCServiceContext; +import org.apache.hyracks.http.api.IServletRequest; +import org.apache.hyracks.http.api.IServletResponse; + +import io.netty.handler.codec.http.HttpResponseStatus; + +/** + * The servlet provides a REST API on an NC for cancelling an on-going query. + */ +public class NCQueryCancellationServlet extends QueryCancellationServlet { +private static final Logger LOGGER = Logger.getLogger(NCQueryCancellationServlet.class.getName()); +private final INCServiceContext serviceCtx; +private final INCMessageBroker messageBroker; + +public NCQueryCancellationServlet(ConcurrentMapctx, String... paths) { +super(ctx, paths); +this.serviceCtx = (INCServiceContext) ctx.get(ServletConstants.SERVICE_CONTEXT_ATTR); +messageBroker = (INCMessageBroker) serviceCtx.getMessageBroker(); +} + +@Override +protected void delete(IServletRequest request, IServletResponse response) throws IOException { +// gets the parameter client_context_id from the request. +String clientContextId = request.getParameter(CLIENT_CONTEXT_ID); +if (clientContextId == null) { +response.setStatus(HttpResponseStatus.BAD_REQUEST); +return; +} +final MessageFuture cancelQueryFuture = messageBroker.registerMessageFuture(); +try { +CancelQueryRequest cancelQueryMessage = +new CancelQueryRequest(serviceCtx.getNodeId(), cancelQueryFuture.getFutureId(), clientContextId); +messageBroker.sendMessageToCC(cancelQueryMessage); +response.setStatus(HttpResponseStatus.OK); +} catch (Exception e) { +if (LOGGER.isLoggable(Level.SEVERE)) { +LOGGER.log(Level.SEVERE, "Unexpected exception while canceling query", e); +} +response.setStatus(HttpResponseStatus.INTERNAL_SERVER_ERROR); +} finally { + messageBroker.deregisterMessageFuture(cancelQueryFuture.getFutureId()); +} +} +} \ No newline at end of file diff --git a/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryCancellationServlet.java b/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryCancellationServlet.java index 3f07151..96ff71c 100644 --- a/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryCancellationServlet.java +++