>From Ian Maxon <[email protected]>:

Attention is currently required from: Peeyush Gupta.

Ian Maxon has posted comments on this change by Peeyush Gupta. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20771?usp=email )

Change subject: [ASTERIXDB-3687][API] Incorrect metrics and missing fields in 
status API
......................................................................


Patch Set 7: Code-Review+2

(1 comment)

File 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java:

https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20771/comment/8a961c73_e1891172?usp=email
 :
PS5, Line 5577:                 
resultMetadata.setCreateTime(System.currentTimeMillis());
> The create time metric is for request creation time and not job creation time.
yeah you're right, i thought this was somehow always pulled out of the hyracks 
side. but it never was, i dug back to trinity and it's always been more or less 
like that. not sure where i got that idea from.



--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20771?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://asterix-gerrit.ics.uci.edu/settings?usp=email

Gerrit-MessageType: comment
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Change-Id: Id5fd38b4273d1a3c98936cea63b8c879e474d619
Gerrit-Change-Number: 20771
Gerrit-PatchSet: 7
Gerrit-Owner: Peeyush Gupta <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Ian Maxon <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Attention: Peeyush Gupta <[email protected]>
Gerrit-Comment-Date: Tue, 13 Jan 2026 17:25:57 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: Peeyush Gupta <[email protected]>
Comment-In-Reply-To: Ian Maxon <[email protected]>

Reply via email to