Clone Tools
  • last updated 15 mins ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[NO ISSUE][HYR] Fix wait for completion work

- user model changes: no

- storage format changes: no

- interface changes: no

details:

- When wait for completion is called on a job that was not created

yet, an exception is returned.

- When wait for completion is called on a job that has been cleared

from job archive, it is retrieved correctly from history.

- When wait for completion is called on a job that has been cleared

from job history, an exception is returned.

- Test cases that fail before the fix have been added.

Change-Id: I9e50f6ce1df9f27517d7ec3a3f8a5d38246f71ff

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1999

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Contrib: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Murtadha Hubail <mhubail@apache.org>

    • -15
    • +28
    ./cc/work/WaitForJobCompletionWork.java
  1. … 5 more files in changeset.
[NO ISSUE][HYR][*DB] Exit JVM from independent thread to avoid deadlocks

Change-Id: I21b2090ea3ef85e95ae90de04b08b4a6d22ebe42

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1973

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Murtadha Hubail <mhubail@apache.org>

    • -23
    • +19
    ./cc/work/ClusterShutdownWork.java
  1. … 7 more files in changeset.
[NO ISSUE] Refactoring / cleanup of HTTP cluster state / diagnostics APIs

Change-Id: If47ec45bf88a39d63421903080fee3ddc0f1e42b

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1969

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Contrib: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Till Westmann <tillw@apache.org>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

  1. … 7 more files in changeset.
[ASTERIXDB-2058][HYR] Only Complete job cancellation after cleanup

- user model changes: no

- storage format changes: no

- interface changes: yes

--IJobManager.cancel now takes a callback

details:

- Job cancellation now completes only after the job cleanup work

has completed and not merely when the abort tasks are executed.

- The NCQueryServiceServlet actively cancels requests that passes

5 minutes.

- Cancellation of timedout jobs is not done through the Http API

but through message broker.

- Typically, requests might timeout when the servers are

overloaded. When that is the case, there is a high chance http

requests are to be rejected including requests to cancel

previously submitted queries. This is the reason for using

Message broker for this task.

- ExecuteStatementRequest used to execute the statement in

a different executor thread even though it is itself is being

executed in an executor thread and is not blocking anyone.

This was fixed as well.

Change-Id: I14b4bbd512cc88e489254d8bf82edba0fd3a3db5

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1961

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Contrib: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Murtadha Hubail <mhubail@apache.org>

  1. … 36 more files in changeset.
[ASTERIXDB-2008][CLUS] Only add pending removal if node known

[ASTERIXDB-2023][ING] Introduce Enums instead of using bytes

- user model changes: no

- storage format changes: no

- interface changes: no

details:

- Only nodes which are known to cluster manager are added

to the list of nodes pending removal. Other nodes are ignored

- Enums introduced:

- ActiveEvent.Kind

- ActivePartitionMessage.Event

- Remove AdapterRuntimeManager

- Remove AdapterExecutor

Change-Id: I7044896559798426c04a3f46861bc5335b25d140

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1921

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Murtadha Hubail <mhubail@apache.org>

    • -0
    • +6
    ./cc/dataset/DatasetDirectoryService.java
  1. … 100 more files in changeset.
[ASTERIXDB-2014][HYR][CLUS] Respect disabled NCService

- Don't contact NCService on failed nodes, if NCService is disabled

- Also, don't block work queue for TriggerNCWork

Change-Id: Ib307f06d8bbcf4039480291aef566f240cadba20

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1913

Reviewed-by: Till Westmann <tillw@apache.org>

Integration-Tests: Till Westmann <tillw@apache.org>

Tested-by: Till Westmann <tillw@apache.org>

[ASTERIXDB-2003][FAIL] Abort jobs failing during job start

- user model changes: no

- storage format changes: no

- interface changes: no

Details:

- Prevent NPE or unmodifiable list in JobCleanupWork and

JobletCleanupNotificationWork.

- Abort job if a failure happens during job start

Change-Id: If6fe4ed9084270f9f22ee4b4c71936d679c8b883

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1904

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Yingyi Bu <buyingyi@gmail.com>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

BAD: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

    • -2
    • +8
    ./cc/work/JobletCleanupNotificationWork.java
[ASTERIXDB-1992][ING] Suspend/Resume active entities

- user model changes: wait for completion only returns

when the entity becomes inactive.

- storage format changes: no

- interface changes:

01) Introduce:

IMetadataLockManager for entity locking.

IActiveNotificationHandler for handling active events.

IRetryPolicy for recovery of failed active jobs.

IActiveEntityController for controlling active entities.

02) IJobLifecycleListener.notifyJobFinish now passes the

JobStatus and Exceptions.

03) IActiveEntityEventsListener.isActive() returns true,

if entity is active, false otherwise.

04) IActiveEntityEventsListener.unregister() removes the

listener upon entity deletion.

05) IActiveEntityEventsListener.getJobFailure() returns

the job failure if the entity is in a failed state.

06) IStatementExecutor.getComponentProvider() returns

the storage component provider.

07) IStatementExecutor.getApplicationContext() returns

the application context.

08) IMetadataManager.upsertEntity to perfrom metadata

entities' upsert operations.

09) IMetadataNode.upsertEntity to perfrom metadata

entities' upsert operations

10) ICcApplicationContext.getMetadataLockManager() returns

the lock manger.

11) ICcApplicationContext.getClusterStateManager() returns

the cluster state manager.

details:

- Starting and stopping of active entities now go through

their listeners rather than having the listener indicating

active entity and not having it indicating inactive entity.

This facilitates suspend/resume operations of long running

active jobs for the sake of DDL operations or topology

changes.

- Unit tests for the vast majority of code paths and

different possible scenarios have been added.

Change-Id: Ifeac8c73e6bad39a13663b84a52121356e3c6b40

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1875

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

BAD: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Murtadha Hubail <mhubail@apache.org>

    • -10
    • +11
    ./cc/ClusterControllerService.java
    • -4
    • +5
    ./cc/application/CCServiceContext.java
    • -3
    • +4
    ./cc/dataset/DatasetDirectoryService.java
  1. … 135 more files in changeset.
[ASTERIXDB-1982][FAIL] Unify runtime error reporting

- user model changes: no

- storage format changes: no

- interface changes: no

Details:

- Unify error reporting in result distribution and operator pipeline;

The fact that ASTERIXDB-1982 is sporadic is because the order of

DatasetDirectoryService.reportJobFailure(...) and

DatasetDirectoryService.reportResultPartitionFailure(...)

is not deterministic and the latter can override the former;

- Make the order of setException and addWaiter irrelevant to

avoid sporadically slipped exception;

- Fix the exception list in Task to be thread-safe.

Change-Id: I36f243c98876ff40e2539ca9241ff6d19fee929f

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1883

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

BAD: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Murtadha Hubail <mhubail@apache.org>

Reviewed-by: Till Westmann <tillw@apache.org>

    • -10
    • +7
    ./cc/dataset/DatasetDirectoryService.java
    • -2
    • +0
    ./cc/dataset/IDatasetDirectoryService.java
    • -51
    • +0
    ./cc/work/ReportResultPartitionFailureWork.java
  1. … 8 more files in changeset.
Re-register NC with CC on reestablished IPCHandle

In case of failed NC -> CC IPCHandle due to CC crash/restart, the NC

needs to re-register with the CC in order to rejoin the cluster, as the

CC ignore heartbeats from unregistered nodes.

- Improve toString on IPCHandle

- Add tests for killing & restarting CC / NCs to NCServiceExecutionIT

- Retrigger NCService on detected dead node

- Ensure jobIds are not reused on CC restart

- NCService shouldn't truncate NC log

Change-Id: I6f93ca9ab37e56e02bafcdecd1e2d0cf664faef6

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1830

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

BAD: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Till Westmann <tillw@apache.org>

  1. … 21 more files in changeset.
ASTERIXDB-1939: No chunked encoding with content-length.

Change-Id: I06d561eb023f1c84c531e9b2cfe88a626d7e5280

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1632

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

BAD: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Michael Blow <mblow@apache.org>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

    • -0
    • +1
    ./cc/web/ApplicationInstallationHandler.java
  1. … 1 more file in changeset.
Attempt to reconnect IPCHandle on connection failure

IPCHandles can become invalid due to network interruption or node

crash/restart. Automatically retry connection in event of attempt

to use disconnected handle.

Change-Id: I069dcd59898021054462c8213fb623df2deec598

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1828

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

BAD: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Yingyi Bu <buyingyi@gmail.com>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

  1. … 6 more files in changeset.
Support IFrameWriter contract check.

- add a instance-level flag for injecting operators to

check IFrameWriter contract violations;

- check contract violations in runtime tests.

Change-Id: I9827b06f640858f27ec1bcca2a39991780bee3b1

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1618

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

BAD: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Yingyi Bu <buyingyi@gmail.com>

  1. … 42 more files in changeset.
Remove -virtual-nc option

Avoid hidden option '-virtual-nc' to indicate when NCService should not

be used to start the NC, instead use ncservice.port of -1 to indicate

the same

Change-Id: I67a9a88808a3d1352b5fdd45ebd158e98dc72dba

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1820

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

BAD: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Till Westmann <tillw@apache.org>

  1. … 7 more files in changeset.
Enable Adding Nodes to Running *DB Cluster

Also ability to configure unique partition ids without having access to

complete cluster topology

Change-Id: If978442a95687c00ef78c89ed1b4440f5e308b99

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1785

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

BAD: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Yingyi Bu <buyingyi@gmail.com>

Reviewed-by: Till Westmann <tillw@apache.org>

  1. … 23 more files in changeset.
Fix Pre-Distributed Jobs

This fix prevents ResultStateSweeper from deinitializing

pre-distributed jobs before they are dropped.

Also fixes issues with new lock manager code

Change-Id: Id50c52fbc7c891761dcabd654fb9b853b5f7a91d

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1656

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

BAD: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Michael Blow <mblow@apache.org>

    • -2
    • +15
    ./cc/dataset/DatasetDirectoryService.java
  1. … 6 more files in changeset.
Add Active Partition Event Message

Enable active runtimes to send messages to the listener. In addition

this change introduces extension locks to metadata lock manager.

Change-Id: I7b4629752e912614927b816d4ce3422ac89c5426

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1596

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

BAD: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: abdullah alamoudi <bamousaa@gmail.com>

  1. … 11 more files in changeset.
Log Message When Applying Logging Level

Log a message when resetting the log level, to indicate when the loggers

are being configured and with what level

Change-Id: I38d939a464cc4ea0e5ec9cca88838f083677cf5e

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1604

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

BAD: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Till Westmann <tillw@apache.org>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

  1. … 5 more files in changeset.
Shutdown ActiveManager Before Killing Thread Executors

Avoid race conditions with start / stop of active runtimes by stopping

the ActiveManager & and started runtimes before terminating the thread

executor

Change-Id: I45e83b0378198f80297fd2741969507741914dea

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1594

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: abdullah alamoudi <bamousaa@gmail.com>

BAD: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

  1. … 14 more files in changeset.
Fix NPE in DatasetDirectoryService.reportJobFailure

Change-Id: I4c27d2133d199bbb6984f8da14e37b2e511528ac

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1582

Reviewed-by: abdullah alamoudi <bamousaa@gmail.com>

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

BAD: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

    • -3
    • +4
    ./cc/dataset/DatasetDirectoryService.java
Deferred result delivery improvements

- Change representation for result set handles to not require URL encoding

- Report errors from the QueryStatusApiServlet for failed jobs

- Fix repeated result consumption for async/deferred requests

- Better error handling in QueryResultApiServlet

- Add host + path to RequestParameters to provide URLs as handles

- Fix logging for failed async jobs

- Fix test framework to not share request parameters between tests

- Move more shared code to AbstractQueryApiServlet

- Make DatasetClientContext a static inner class of HyracksDataset

Change-Id: I88fe289fe9109ea012c63d82af0083dce6bde31b

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1548

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Michael Blow <mblow@apache.org>

BAD: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

    • -11
    • +28
    ./cc/dataset/DatasetDirectoryService.java
  1. … 75 more files in changeset.
Rename Class Helper and cleanup ICCApplication interface

Change-Id: I2f6deb10ff240e85b26244b52c99776c0d1d3291

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1567

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Michael Blow <mblow@apache.org>

  1. … 5 more files in changeset.
Unify structure of Cluster Controller and Node Controller

Change-Id: Ife3a002371a07ee9cdd32e1ffd50cc775bf1d453

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1514

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Michael Blow <mblow@apache.org>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

    • -0
    • +79
    ./cc/BaseCCApplication.java
    • -24
    • +29
    ./cc/ClusterControllerService.java
    • -125
    • +0
    ./cc/application/CCApplicationContext.java
    • -0
    • +130
    ./cc/application/CCServiceContext.java
  1. … 146 more files in changeset.
Add Parameter To Control Job Queue Capacity

Exposed CC property job.queue.capacity (-job-queue-capacity) to enable

ability to override default capacity of 4k

Change-Id: I36d0727de58dbe9697e3693e49b39f8c8ab32ce8

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1562

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

BAD: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Yingyi Bu <buyingyi@gmail.com>

  1. … 2 more files in changeset.
Cleanup and bug fixes in Feeds pipeline

The bug fix is for the MessagingFrameTupleAppender. It used

to consume one extra byte per frame.

Cleanups are for feeds. These include:

1. Remove the use of feed dataflow marker. Feeds which

need to send progress can and should do that without

a marker thread.

2. Lock the memory component for feed commit to be able

to add information to the memory component's metadata

page safely.

In addition, this change introduces a frame level callback

for index operations.

Change-Id: Ie97b2133ebecb7380cf0ba336e60ed714d06f8ee

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1523

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

BAD: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Yingyi Bu <buyingyi@gmail.com>

  1. … 72 more files in changeset.
Configuration Revamp

- Ini section of node / cc details now returns ini param names instead of

managix option names

- Normalized command line -vs- ini file configuration parameter names

- Eliminated unused parameters

- Ini validation

- Migrate *DB parameters out of [app] and into nc / cc sections as

appropriate

- Eliminate [app] section. Cluster-wide configuration lives in [common]

- Sort properties alphabetically when returned by HTTP api

Change-Id: I95b7e0bd4538ef42817c8826e76412150074b754

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1487

Reviewed-by: Michael Blow <mblow@apache.org>

Integration-Tests: Michael Blow <mblow@apache.org>

Tested-by: Michael Blow <mblow@apache.org>

    • -0
    • +68
    ./cc/CCApplicationEntryPoint.java
    • -69
    • +74
    ./cc/ClusterControllerService.java
    • -2
    • +3
    ./cc/application/CCApplicationContext.java
    • -42
    • +17
    ./cc/work/GetNodeDetailsJSONWork.java
  1. … 165 more files in changeset.
Add job cancellation support in Hyracks.

This change also:

- cleans up JobRun;

- adds tests for job cancellation.

Change-Id: Ic26330c19c8642dd3246739b5150c4aa667c359c

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1537

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

BAD: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Till Westmann <tillw@apache.org>

    • -0
    • +53
    ./cc/work/CancelJobWork.java
  1. … 14 more files in changeset.
Add links to node/job details to summaries

Change-Id: I012acbf10eb6f11a625a2a5e51eed1421d8fd046

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1530

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: abdullah alamoudi <bamousaa@gmail.com>

    • -4
    • +4
    ./cc/web/ApplicationInstallationHandler.java
    • -1
    • +1
    ./cc/web/StateDumpRESTAPIFunction.java
    • -1
    • +1
    ./cc/web/util/IJSONOutputFunction.java
    • -8
    • +10
    ./cc/web/util/JSONOutputRequestHandler.java
    • -0
    • +39
    ./cc/web/util/JSONOutputRequestUtil.java
  1. … 8 more files in changeset.
Handle error conditions in the /query/result API

- introduce AbstractQueryApiServlet to factor out common code of

QueryResultApiServlet and QueryStatusApiServlet

- clean up ResultReader interface

- remove org.apache.asterix.api.http.servlet.HyracksProperties

Change-Id: Icb99fccb4b41768fa010c574bf1703ffcd47535e

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1532

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Yingyi Bu <buyingyi@gmail.com>

    • -17
    • +16
    ./cc/dataset/DatasetDirectoryService.java
  1. … 16 more files in changeset.
Handle error conditions in the /query/status API

Change-Id: I30176c5c70dcc5f7f6605ad79dd0e41967373d9c

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1522

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Yingyi Bu <buyingyi@gmail.com>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

    • -7
    • +3
    ./cc/dataset/DatasetDirectoryService.java
  1. … 15 more files in changeset.