Change in asterixdb[master]: Add result location HTTP API.
Wail Alkowaileet has abandoned this change. Change subject: Add result location HTTP API. .. Abandoned New patch has been submitted... -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: abandon Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 11 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 11: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/1924/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 11 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 11: BAD-1 BAD Compatibility Tests Failed https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/567/ : FAILURE -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 11 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 11: BAD Compatibility Tests Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/567/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 11 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 11: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/1924/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 11 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 11: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/4330/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 11 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1178 to look at the new patch set (#11). Change subject: Add result location HTTP API. .. Add result location HTTP API. This addition allows other systems to connect to AsterixDB and collect the result locations of a an async query. Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 --- A asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryResultLocationsAPIServlet.java M asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplicationEntryPoint.java M asterixdb/asterix-app/src/test/resources/runtimets/queries/api/APIQueries.xml A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_1/result_location_1.1.async.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_1/result_location_1.2.pollget.http A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_1/result_location_1.3.get.http A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.1.ddl.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.2.update.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.3.async.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.4.pollget.http A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.5.get.http A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_1/result_location_1.1.ignore A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_1/result_location_1.2.json A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_1/result_location_1.3.regexadm A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_2/result_location_2.3.ignore A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_2/result_location_2.4.json A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_2/result_location_2.5.regexadm M asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/Servlets.java M asterixdb/asterix-common/src/test/java/org/apache/asterix/test/aql/TestExecutor.java 19 files changed, 423 insertions(+), 16 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/78/1178/11 -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 11 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 10: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/1865/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 10 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 10: BAD-1 BAD Compatibility Tests Failed https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/523/ : FAILURE -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 10 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 10: BAD Compatibility Tests Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/523/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 10 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 10: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/1865/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 10 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 10: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/4263/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 10 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1178 to look at the new patch set (#10). Change subject: Add result location HTTP API. .. Add result location HTTP API. This addition allows other systems to connect to AsterixDB and collect the result locations of a an async query. Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 --- A asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryResultLocationsAPIServlet.java M asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplicationEntryPoint.java M asterixdb/asterix-app/src/test/resources/runtimets/queries/api/APIQueries.xml A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_1/result_location_1.1.async.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_1/result_location_1.2.pollget.http A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_1/result_location_1.3.get.http A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.1.ddl.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.2.update.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.3.async.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.4.pollget.http A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.5.get.http A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_1/result_location_1.1.ignore A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_1/result_location_1.2.json A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_1/result_location_1.3.regexadm A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_2/result_location_2.3.ignore A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_2/result_location_2.4.json A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_2/result_location_2.5.regexadm M asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/LetUtil.java 18 files changed, 416 insertions(+), 2 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/78/1178/10 -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 10 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Wail Alkowaileet
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 9: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/4262/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 9 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1178 to look at the new patch set (#9). Change subject: Add result location HTTP API. .. Add result location HTTP API. This addition allows other systems to connect to AsterixDB and collect the result locations of a an async query. Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 --- A asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryResultLocationsAPIServlet.java M asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplicationEntryPoint.java M asterixdb/asterix-app/src/test/resources/runtimets/queries/api/APIQueries.xml A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_1/result_location_1.1.async.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_1/result_location_1.2.pollget.http A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_1/result_location_1.3.get.http A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.1.ddl.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.2.update.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.3.async.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.4.pollget.http A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.5.get.http A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_1/result_location_1.1.ignore A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_1/result_location_1.2.json A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_1/result_location_1.3.regexadm A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_2/result_location_2.3.ignore A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_2/result_location_2.4.json A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_2/result_location_2.5.regexadm M asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/LetUtil.java M asterixdb/asterix-common/src/test/java/org/apache/asterix/test/aql/TestExecutor.java 19 files changed, 479 insertions(+), 33 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/78/1178/9 -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 9 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Wail Alkowaileet
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 8: Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/1843/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 8 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 8: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/1842/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 8 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 8: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/1843/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 8 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 8: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/1842/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 8 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 8: Integration-Tests-1 Integration Tests Timed Out https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/1826/ : ABORTED -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 8 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 8: BAD+1 BAD Compatibility Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/494/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 8 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 8: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/1826/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 8 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 8: BAD Compatibility Tests Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/494/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 8 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 8: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/4213/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 8 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1178 to look at the new patch set (#8). Change subject: Add result location HTTP API. .. Add result location HTTP API. This addition allows other systems to connect to AsterixDB and collect the result locations of a an async query. Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 --- A asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryResultLocationsAPIServlet.java M asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplicationEntryPoint.java M asterixdb/asterix-app/src/test/resources/runtimets/queries/api/APIQueries.xml A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_1/result_location_1.1.rlocation.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.1.ddl.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.2.update.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.3.rlocation.aql A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_1/result_location_1.1.regexadm A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_2/result_location_1.3.regexadm M asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/LetUtil.java M asterixdb/asterix-common/src/test/java/org/apache/asterix/test/aql/TestExecutor.java 11 files changed, 372 insertions(+), 27 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/78/1178/8 -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 8 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Wail Alkowaileet
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 7: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/1825/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 7 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 7: BAD+1 BAD Compatibility Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/493/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 7 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 7: BAD Compatibility Tests Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/493/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 7 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 7: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/1825/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 7 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 7: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/4212/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 7 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1178 to look at the new patch set (#7). Change subject: Add result location HTTP API. .. Add result location HTTP API. This addition allows other systems to connect to AsterixDB and collect the result locations of a an async query. Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 --- A asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryResultLocationsAPIServlet.java M asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplicationEntryPoint.java M asterixdb/asterix-app/src/test/resources/runtimets/queries/api/APIQueries.xml A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_1/result_location_1.1.rlocation.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.1.ddl.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.2.update.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.3.rlocation.aql A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_1/result_location_1.1.regexadm A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_2/result_location_1.3.regexadm M asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/LetUtil.java M asterixdb/asterix-common/src/test/java/org/apache/asterix/test/aql/TestExecutor.java 11 files changed, 368 insertions(+), 27 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/78/1178/7 -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 7 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Wail Alkowaileet
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 6: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/4211/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 6 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1178 to look at the new patch set (#6). Change subject: Add result location HTTP API. .. Add result location HTTP API. This addition allows other systems to connect to AsterixDB and collect the result locations of a an async query. Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 --- A asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryResultLocationsAPIServlet.java M asterixdb/asterix-app/src/main/java/org/apache/asterix/app/external/ExternalLibraryUtils.java M asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplicationEntryPoint.java M asterixdb/asterix-app/src/test/resources/runtimets/queries/api/APIQueries.xml A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_1/result_location_1.1.rlocation.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.1.ddl.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.2.update.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.3.rlocation.aql A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_1/result_location_1.1.regexadm A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_2/result_location_1.3.regexadm M asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/LetUtil.java M asterixdb/asterix-common/src/test/java/org/apache/asterix/test/aql/TestExecutor.java M asterixdb/asterix-experiments/pom.xml 13 files changed, 373 insertions(+), 32 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/78/1178/6 -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 6 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Wail Alkowaileet
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 5: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/4210/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 5 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1178 to look at the new patch set (#5). Change subject: Add result location HTTP API. .. Add result location HTTP API. This addition allows other systems to connect to AsterixDB and collect the result locations of a an async query. Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 --- A asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryResultLocationsAPIServlet.java M asterixdb/asterix-app/src/main/java/org/apache/asterix/app/external/ExternalLibraryUtils.java M asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplicationEntryPoint.java M asterixdb/asterix-app/src/test/resources/runtimets/queries/api/APIQueries.xml A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_1/result_location_1.1.rlocation.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.1.ddl.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.2.update.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.3.rlocation.aql A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_1/result_location_1.1.regexadm A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_2/result_location_1.3.regexadm M asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/LetUtil.java M asterixdb/asterix-common/src/test/java/org/apache/asterix/test/aql/TestExecutor.java D asterixdb/asterix-doc/pom.xml M asterixdb/asterix-experiments/pom.xml 14 files changed, 373 insertions(+), 138 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/78/1178/5 -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 5 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet
Change in asterixdb[master]: Add result location HTTP API.
Michael Blow has posted comments on this change.
Change subject: Add result location HTTP API.
..
Patch Set 4:
(2 comments)
https://asterix-gerrit.ics.uci.edu/#/c/1178/4/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryResultLocationsAPIServlet.java
File
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryResultLocationsAPIServlet.java:
Line 104: status = "ERROR";
Are there other enum values defined? If not, should this be a runtime
exception (e.g. IllegalStateException("NYI: " + resultReader.getStatus())?
Line 109: prepareLocations(jsonResponse, locs);
I wonder if this was moved ahead of the switch above, the logic in the
following lines could be simplified- the cases could just do
jsonResponse.put("status", "RUNNING"), etc.
and there wouldn't be the need to special case SUCCESS & not all ready, it
could just be added to the same case block as RUNNING? just a thought.
--
To view, visit https://asterix-gerrit.ics.uci.edu/1178
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wail Alkowaileet
Gerrit-Reviewer: Jenkins
Gerrit-Reviewer: Michael Blow
Gerrit-Reviewer: Till Westmann
Gerrit-Reviewer: Wail Alkowaileet
Gerrit-HasComments: Yes
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 4: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/1030/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 4: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/1030/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Michael Blow has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 4: TRIGGER: asterix-gerrit-integration-tests -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 4: Integration Tests Failed https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/1005/ : UNSTABLE -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 4: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/1005/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 4: Integration-Tests-1 Integration Tests Timed Out https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/1003/ : ABORTED -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 4: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/1003/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Wail Alkowaileet has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 3: Sorry for my ultimate stupid mistakes :-) I added to types of error handling: 1- for invalid handle format. 2- generic exception. I directed the user to check of the logs. Should other "async" APIs report errors similarly ? Currently we just do: out.println(e.getMessage()); e.printStackTrace(out); P.S. I filed a JIRA for a bug. Requesting unexisted jobId will halt the whole query api from responding. -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 4: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/3122/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1178 to look at the new patch set (#4). Change subject: Add result location HTTP API. .. Add result location HTTP API. This addition allows other systems to connect to AsterixDB and collect the result locations of a an async query. Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 --- A asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryResultLocationsAPIServlet.java M asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplicationEntryPoint.java M asterixdb/asterix-app/src/test/resources/runtimets/queries/api/APIQueries.xml A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_1/result_location_1.1.rlocation.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.1.ddl.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.2.update.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.3.rlocation.aql A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_1/result_location_1.1.regexadm A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_2/result_location_1.3.regexadm M asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/ServletUtil.java M asterixdb/asterix-common/src/test/java/org/apache/asterix/test/aql/TestExecutor.java 11 files changed, 376 insertions(+), 30 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/78/1178/4 -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet
Change in asterixdb[master]: Add result location HTTP API.
Michael Blow has posted comments on this change.
Change subject: Add result location HTTP API.
..
Patch Set 3:
(3 comments)
https://asterix-gerrit.ics.uci.edu/#/c/1178/3/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryResultLocationsAPIServlets.java
File
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryResultLocationsAPIServlets.java:
Line 47: public class QueryResultLocationsAPIServlets extends HttpServlet {
same comment as patch 2 (/Servlets/Servlet/)
Line 119: GlobalConfig.ASTERIX_LOGGER.log(Level.SEVERE,
e.getMessage(), e);
e.getMessage() is often not useful for many types of exception. Given that the
exception itself is also being logged, perhaps just some message "Unexpected
exception"?
https://asterix-gerrit.ics.uci.edu/#/c/1178/3/asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_1/result_location_1.1.rlocation.aql
File
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_1/result_location_1.1.rlocation.aql:
Line 25:
WS
--
To view, visit https://asterix-gerrit.ics.uci.edu/1178
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wail Alkowaileet
Gerrit-Reviewer: Jenkins
Gerrit-Reviewer: Michael Blow
Gerrit-Reviewer: Till Westmann
Gerrit-Reviewer: Wail Alkowaileet
Gerrit-HasComments: Yes
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 3: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/994/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Wail Alkowaileet has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 3: Thanks Michael B. -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 3: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/994/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/3105/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1178 to look at the new patch set (#3). Change subject: Add result location HTTP API. .. Add result location HTTP API. This addition allows other systems to connect to AsterixDB and collect the result locations of a an async query. Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 --- A asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryResultLocationsAPIServlets.java M asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplicationEntryPoint.java M asterixdb/asterix-app/src/test/resources/runtimets/queries/api/APIQueries.xml A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_1/result_location_1.1.rlocation.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.1.ddl.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.2.update.aql A asterixdb/asterix-app/src/test/resources/runtimets/queries/api/result_location_2/result_location_2.3.rlocation.aql A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_1/result_location_1.1.regexadm A asterixdb/asterix-app/src/test/resources/runtimets/results/api/result_location_2/result_location_1.3.regexadm M asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/ServletUtil.java M asterixdb/asterix-common/src/test/java/org/apache/asterix/test/aql/TestExecutor.java 11 files changed, 365 insertions(+), 30 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/78/1178/3 -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet
Change in asterixdb[master]: Add result location HTTP API.
Michael Blow has posted comments on this change.
Change subject: Add result location HTTP API.
..
Patch Set 2:
(1 comment)
Hi, see https://asterix-gerrit.ics.uci.edu/#/c/1170/ for examples of two ways
variable results are now suported. for regexadm result files, there is a
line-by-line regex match. For regex result files, each line of the regex file
is matched against the entirety of the actual result. A - in front of the
expression indicates that a match in the actual result is forbidden, and should
fail if matched. Hope this helps...
https://asterix-gerrit.ics.uci.edu/#/c/1178/2/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryResultLocationsAPIServlets.java
File
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryResultLocationsAPIServlets.java:
Line 44: public class QueryResultLocationsAPIServlets extends HttpServlet {
Should this be just *Servlet instead of *Servlets?
--
To view, visit https://asterix-gerrit.ics.uci.edu/1178
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wail Alkowaileet
Gerrit-Reviewer: Jenkins
Gerrit-Reviewer: Michael Blow
Gerrit-Reviewer: Till Westmann
Gerrit-Reviewer: Wail Alkowaileet
Gerrit-HasComments: Yes
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 2: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/801/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 2: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/801/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1178 to look at the new patch set (#2). Change subject: Add result location HTTP API. .. Add result location HTTP API. This addition allows other systems to connect to AsterixDB and collect the result locations of a an async query. Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 --- A asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryResultLocationsAPIServlets.java M asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplicationEntryPoint.java M asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/ServletUtil.java 3 files changed, 160 insertions(+), 0 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/78/1178/2 -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2856/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Till Westmann has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 1: For the test: I think that Michael Blow recently added some regular expression support to the test framework when he was writing tests for the cluster admin APIs. I think that that might help. For exceptions: I think that the practice to ignore exceptions that we don't know how to handle is quite dangerous. I think that in many of the cases where that is done, we don't understand the impact on the system behavior (and if we do, we should document it). So I think that we need to fix that (either immediately or later by filing an issue) one handler at a time ... -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Wail Alkowaileet has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 1: 2) Nevermind the issue about the Life Cycle Management. It seems the subsequent queries work even after the exception. -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Wail Alkowaileet has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 1: For the error message: I see other APIs are only print the error and the stack trace and not producing error messages? I used to call the methods in ResultUtil. I was just following other APIs. Should I file a JIRA for that? or that expected ? -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Wail Alkowaileet has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 1: Hi Till, 1) Unfortunately the port is not deterministic. I generated the test result file. However it seems the port is determined on runtime to get a free port. Any suggestion for that? 2) Sure will do that. One thing I noticed is that it throws exception and stops Life Cycle Management if the result got swept and read after sweeping. I thinks should not affect the whole Life Cycle Management? -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wail Alkowaileet Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Till Westmann has posted comments on this change.
Change subject: Add result location HTTP API.
..
Patch Set 1:
(4 comments)
I've got a few comments in the source and a few requests :)
1) Could you add a test?
2) Could you file an issue about the weak error handling capabilities of the
result distribution framework that mentions that
a) HyracksDatasetReader.getResultStatus ignores exceptions and
b) DatasetDirectoryService.getResultStatus hangs forever on an invalid jobId?
https://asterix-gerrit.ics.uci.edu/#/c/1178/1/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryResultLocationsAPIServlets.java
File
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryResultLocationsAPIServlets.java:
Line 14: */
Please use an Apache license header.
Line 56: String strHandle = request.getParameter("handle");
Handle the case if no handle is given. Should probably return a 400.
Line 81: JSONArray handle = handleObj.getJSONArray("handle");
Check the syntax of the handle.
Line 116: out.println(e.getMessage());
Could we produce a JSON error like
{ "errors": [ { "msg" : "bla bla" } ] }
here?
--
To view, visit https://asterix-gerrit.ics.uci.edu/1178
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wail Alkowaileet
Gerrit-Reviewer: Jenkins
Gerrit-Reviewer: Till Westmann
Gerrit-HasComments: Yes
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 1: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/650/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 1: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/650/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
Change in asterixdb[master]: Add result location HTTP API.
Wail Alkowaileet has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/1178
Change subject: Add result location HTTP API.
..
Add result location HTTP API.
This addition allows other systems to connect to AsterixDB
and collect the result locations of a an async query.
Change-Id: If5d691635c6003947b4eb06291f620127de1eef8
---
A
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryResultLocationsAPIServlets.java
M
asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplicationEntryPoint.java
M
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/ServletUtil.java
3 files changed, 160 insertions(+), 0 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/78/1178/1
diff --git
a/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryResultLocationsAPIServlets.java
b/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryResultLocationsAPIServlets.java
new file mode 100644
index 000..da0eaa2
--- /dev/null
+++
b/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryResultLocationsAPIServlets.java
@@ -0,0 +1,155 @@
+/*
+ * Copyright 2009-2013 by The Regents of the University of California
+ * Licensed 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 from
+ *
+ * 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.servlet;
+
+import java.io.IOException;
+import java.io.PrintWriter;
+
+import javax.servlet.ServletContext;
+import javax.servlet.http.HttpServlet;
+import javax.servlet.http.HttpServletRequest;
+import javax.servlet.http.HttpServletResponse;
+
+import org.apache.asterix.app.result.ResultReader;
+import org.apache.hyracks.api.client.HyracksConnection;
+import org.apache.hyracks.api.client.IHyracksClientConnection;
+import org.apache.hyracks.api.comm.NetworkAddress;
+import org.apache.hyracks.api.dataset.DatasetDirectoryRecord;
+import org.apache.hyracks.api.dataset.DatasetJobRecord.Status;
+import org.apache.hyracks.api.dataset.IHyracksDataset;
+import
org.apache.hyracks.api.dataset.IHyracksDatasetDirectoryServiceConnection;
+import org.apache.hyracks.api.dataset.ResultSetId;
+import org.apache.hyracks.api.job.JobId;
+import org.apache.hyracks.client.dataset.HyracksDataset;
+import
org.apache.hyracks.client.dataset.HyracksDatasetDirectoryServiceConnection;
+import org.json.JSONArray;
+import org.json.JSONException;
+import org.json.JSONObject;
+
+/**
+ * A servlet to get the result locations.
+ */
+public class QueryResultLocationsAPIServlets extends HttpServlet {
+
+private static final long serialVersionUID = 1L;
+
+private static final String HYRACKS_CONNECTION_ATTR =
"org.apache.asterix.HYRACKS_CONNECTION";
+
+private static final String HYRACKS_DATASET_ATTR =
"org.apache.asterix.HYRACKS_DATASET";
+
+@Override
+public void doGet(HttpServletRequest request, HttpServletResponse
response) throws IOException {
+response.setContentType("text/html");
+response.setCharacterEncoding("utf-8");
+String strHandle = request.getParameter("handle");
+PrintWriter out = response.getWriter();
+ServletContext context = getServletContext();
+IHyracksClientConnection hcc;
+IHyracksDataset hds;
+
+try {
+HyracksProperties hp = new HyracksProperties();
+String strIP = hp.getHyracksIPAddress();
+int port = hp.getHyracksPort();
+
+synchronized (context) {
+hcc = (IHyracksClientConnection)
context.getAttribute(HYRACKS_CONNECTION_ATTR);
+if (hcc == null) {
+hcc = new HyracksConnection(strIP, port);
+context.setAttribute(HYRACKS_CONNECTION_ATTR, hcc);
+}
+
+hds = (IHyracksDataset)
context.getAttribute(HYRACKS_DATASET_ATTR);
+if (hds == null) {
+hds = new HyracksDataset(hcc, ResultReader.FRAME_SIZE,
ResultReader.NUM_READERS);
+context.setAttribute(HYRACKS_DATASET_ATTR, hds);
+}
+}
+JSONObject handleObj = new JSONObject(strHandle);
+JSONArray handle = handleObj.getJSONArray("handle");
+JobId jobId = new JobId(handle.getLong(0));
+ResultSetId rsId = new ResultSetId(handle.getLong(1));
+
+ResultReader resultRead
Change in asterixdb[master]: Add result location HTTP API.
Jenkins has posted comments on this change. Change subject: Add result location HTTP API. .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2656/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1178 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5d691635c6003947b4eb06291f620127de1eef8 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
