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>