>From Suryaa Charan Shivakumar <[email protected]>: Attention is currently required from: Ian Maxon. Suryaa Charan Shivakumar 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 3: (3 comments) Commit Message: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19390/comment/1def001c_8bc9296d 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 I did add spaces and new line character Patchset: PS3: Added the test cases and fixed the optimized logical plan issue 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/85fdfd9f_b9c8e08e PS2, Line 25: -- param logical-plan:string=true > let's compare the optimized logical plan, the unoptimized is seldom used Added in compileonly.8 test file -- 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: 3 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: Ian Maxon <[email protected]> Gerrit-Comment-Date: Fri, 14 Mar 2025 00:28:12 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Ian Maxon <[email protected]> Gerrit-MessageType: comment
