>From Ian Maxon <[email protected]>:

Attention is currently required from: Suryaa Charan Shivakumar.
Ian Maxon has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19390 )

Change subject: [NO ISSUE] Added support for printing logical plan in DOT 
format  - Added support in UI and API(s) Change-Id: 
Ia6e37080a581292744ddd9030b294936513c15ac
......................................................................


Patch Set 2:

(2 comments)

Commit Message:

https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19390/comment/11b74663_9b1fc023
PS2, Line 7: [NO ISSUE] Added support for printing logical plan in DOT format
there should be spaces after each line otherwise it gets all smushed into 1 in 
gerrit


File 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/api/compileonly/compileonly.6.plans.sqlpp:

https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19390/comment/e28205ac_fb2f0fcc
PS2, Line 25: -- param logical-plan:string=true
let's compare the optimized logical plan, the unoptimized is seldom used



--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19390
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: Ia6e37080a581292744ddd9030b294936513c15ac
Gerrit-Change-Number: 19390
Gerrit-PatchSet: 2
Gerrit-Owner: Suryaa Charan Shivakumar <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-CC: Ian Maxon <[email protected]>
Gerrit-Attention: Suryaa Charan Shivakumar <[email protected]>
Gerrit-Comment-Date: Thu, 13 Mar 2025 16:57:51 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment

Reply via email to