>From Ian Maxon <[email protected]>:

Ian Maxon has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19809 )

Change subject: [ASTERIXDB-3611][RT] Add more stats to query profile
......................................................................


Patch Set 3:

(1 comment)

File 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/prettyprint/LogicalOperatorPrettyPrintVisitorJson.java:

https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19809/comment/860cdb58_de8bd635
PS1, Line 283: private static class WeightedAvg
> AI is suggesting this: […]
yea thats more straightforward. i was just kind of copying the formula verbatim 
because this only gets executed for however many partitions you have. i removed 
the getter since the value is always valid.



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

Gerrit-Project: asterixdb
Gerrit-Branch: ionic
Gerrit-Change-Id: I524cbfb1685ad3c7ed269c2db48482e5aba1ce61
Gerrit-Change-Number: 19809
Gerrit-PatchSet: 3
Gerrit-Owner: Ian Maxon <[email protected]>
Gerrit-Reviewer: Ali Alsuliman <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Ian Maxon <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Comment-Date: Tue, 01 Jul 2025 15:58:51 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Ali Alsuliman <[email protected]>
Gerrit-MessageType: comment

Reply via email to