>From Ian Maxon <[email protected]>:

Attention is currently required from: Ali Alsuliman, Vijay Sarathy, 
[email protected].
Ian Maxon has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17428 )

Change subject: [ASTERIXDB-3143] Add SqlppAnalyzedExecutionTest
......................................................................


Patch Set 7: Code-Review+1

(1 comment)

File 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/join/hash-join-with-redundant-variable/hash-join-with-redundant-variable.08.plan:

https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17428/comment/b09d11a5_5624da04
PS6, Line 1: distribute result [$$38] [cardinality: 0.0, op-cost: 0.0, 
total-cost: 0.0]
> This file is the same as the one in "results". Please check if this needs to 
> be updated. […]
good catch, yeah this just got added on accident there's no delta. works fine 
without it.



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

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Change-Id: Ib5e1b2efefe38332ea1f8f7328c63ace12ac9cab
Gerrit-Change-Number: 17428
Gerrit-PatchSet: 7
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-Reviewer: Vijay Sarathy <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Attention: Ali Alsuliman <[email protected]>
Gerrit-Attention: Vijay Sarathy <[email protected]>
Gerrit-Attention: [email protected]
Gerrit-Comment-Date: Mon, 10 Apr 2023 21:52:22 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: Vijay Sarathy <[email protected]>
Gerrit-MessageType: comment

Reply via email to