Hi Suryaa,

Thanks for your review — responses to your thoughts are below.

Answer 1:
Yes, this is what I'm implementing in the current patch. We will assign 
sequential statement IDs using `statements.indexOf(stmt)` in the CC, without 
requiring any changes to query parsing logic.

Answer 2:
The response and statements are sequential. In Example 1, we have four 
statements. The response shows statementIDs from 1 to 4 corresponding to these 
four statements in order. We can include the corresponding statement text in 
the response in a future update.

Answer 3:
Moving forward with the new API structure provides a cleaner and more efficient 
implementation for executing multiple statements. This new structure aligns 
well with AsterixDB Dashboard's needs. When executing multiple statements from 
the UI, the new API structure provides individual metrics for each statement, 
which can be leveraged in the future.

Thanks,
Janhavi

From: Suryaa Charan Shivakumar <sshiv...@ucr.edu.INVALID>
Date: Friday, 11 April 2025 at 11:04 PM
To: dev@asterixdb.apache.org <dev@asterixdb.apache.org>
Subject: Re: [DISCUSS][APE] Proposal for Support Individual Responses for 
Multi-Statement Queries
Hello Janhavi,

This would be a useful feature to have. Here are some thoughts/suggestions,
1. (id: int) - can this be handled by the CC somewhere within asterixDB if
multi-statement is enabled in request? Since we have a unique requestID we
can assign random/sequential ID for statements I believe. This may help the
queries look cleaner and not add changes to query parsing logic.
2. Not sure if this is intentional, in example 1 the statement ID in
response is not in-line with example 1 query.
3. Backward Compatibility - I'd suggest adding additional parameters in the
request body like "multi-statement" and additional response objects without
touching the existing API structure. Or using versioning for the API.

Overall, it is a great feature to have. +1 to that!

Best,
Suryaa

Reply via email to