You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@asterixdb.apache.org by AsterixDB Code Review <do...@asterix-gerrit.ics.uci.edu> on 2026/08/09 15:28:20 UTC

[L] Change in asterixdb[master]: [ASTERIXDB-3593][UI]: Support multi-statement responses in dashboard

From Janhavi Tripurwar <ja...@apache.org>:

Attention is currently required from: Ali Alsuliman, Ian Maxon, Janhavi Tripurwar, Murtadha Hubail.

Hello Ali Alsuliman, Anon. E. Moose #1000171, Ian Maxon, Jenkins, Murtadha Hubail, 

I'd like you to reexamine a change. Please visit

    https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19689?usp=email

to look at the new patch set (#7).

The following approvals got outdated and were removed:
Code-Review+1 by Janhavi Tripurwar, Code-Review+2 by Ian Maxon, Integration-Tests+1 by Jenkins, Verified-1 by Anon. E. Moose #1000171, Verified-1 by Jenkins


Change subject: [ASTERIXDB-3593][UI]: Support multi-statement responses in dashboard
......................................................................

[ASTERIXDB-3593][UI]: Support multi-statement responses in dashboard

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

details:
- The response shape follows the request's content, so the dashboard
  has to read both: flat for one real statement, statements[] for
  more. The new shared/utils/response.utils.ts reads a flat response
  as its own single statement, and every consumer goes through it.
- Single-statement responses are byte-identical, so every request the
  dashboard issues on its own keeps taking the flat path unchanged.
- One result view: the first statement that returned rows, falling
  back to the first carrying plans. Metrics are aggregated with
  elapsedTime as the max, since it is measured from request start,
  and the rest summed; durations are parsed unit-aware and formatted
  the way Duration.formatNanos does.
- A statement failing partway through a batch returns HTTP 200, so
  its errors are lifted out of statements[] to the error path.

Co-Authored-By: Claude Opus 5 (1M context) <no...@anthropic.com>
Change-Id: I78deab5ec0f43798d7c69dc7610f8accda18d56f
---
M asterixdb/asterix-dashboard/src/node/src/app/dashboard/query/input.component.ts
M asterixdb/asterix-dashboard/src/node/src/app/dashboard/query/metadata.component.ts
M asterixdb/asterix-dashboard/src/node/src/app/dashboard/query/output.component.ts
M asterixdb/asterix-dashboard/src/node/src/app/shared/reducers/dataset.reducer.ts
M asterixdb/asterix-dashboard/src/node/src/app/shared/reducers/query.reducer.ts
M asterixdb/asterix-dashboard/src/node/src/app/shared/services/async-query.service.ts
A asterixdb/asterix-dashboard/src/node/src/app/shared/utils/response.utils.ts
7 files changed, 276 insertions(+), 30 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/89/19689/7
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19689?usp=email
To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings?usp=email

Gerrit-MessageType: newpatchset
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Change-Id: I78deab5ec0f43798d7c69dc7610f8accda18d56f
Gerrit-Change-Number: 19689
Gerrit-PatchSet: 7
Gerrit-Owner: Janhavi Tripurwar <ja...@apache.org>
Gerrit-Reviewer: Ali Alsuliman <al...@gmail.com>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Janhavi Tripurwar <ja...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Murtadha Hubail <mh...@apache.org>
Gerrit-Attention: Murtadha Hubail <mh...@apache.org>
Gerrit-Attention: Janhavi Tripurwar <ja...@apache.org>
Gerrit-Attention: Ian Maxon <im...@apache.org>
Gerrit-Attention: Ali Alsuliman <al...@gmail.com>