Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FRG-8] show cost in EXPLAIN PLAN, and allow for display of other derived attributes #862

Open
dynamobi-build opened this issue Dec 30, 2011 · 4 comments

Comments

@dynamobi-build
Copy link

[reporter="jvs", created="Wed, 21 Dec 2005 10:41:56 -0500 (GMT-05:00)"]
Need to allow rels to flag whether each attributes should be an input to the digest; purely informational attributes such as cost should NOT be inputs to the digest.



Even though we have an XML format, probably still need a detail parameter controlling how much information is displayed, since sometimes we care about cost in diff-based tests, but often we don't.

@dynamobi-build
Copy link
Author

[author="jvs", created="Fri, 23 Dec 2005 11:22:09 -0500 (GMT-05:00)"]
In eigenchange 4814, I implemented new syntax:



EXPLAIN PLAN

[ { EXCLUDING | INCLUDING [ ALL ] } ATTRIBUTES ]

[ { WITH | WITHOUT } IMPLEMENTATION ]

[ AS XML ]

FOR query-or-DML-statement



The default detail level is INCLUDING ATTRIBUTES. EXCLUDING ATTRIBUTES displays only operator names, while INCLUDING ALL ATTRIBUTES adds additional attributes (currently just cost).



Now, theoretically if rels wanted to tack on extra information, they can test with "if planWriter.getDetailLevel() == SqlExplainLevel.ALL_ATTRIBUTES)", and then pass longer arrays. However, the array-based mechanism for properties is confusing (mismatching array lengths based on number of children) and difficult to use once optional attributes are possible. Also, the XML plan writer is a little world of its own, so I just left a TODO there. So I'm leaving this issue until we enhance the internal interface.

@dynamobi-build
Copy link
Author

[author="jvs", created="Tue, 27 Dec 2005 14:23:51 -0500 (GMT-05:00)"]
Changing priority to minor since display of cost was the most important thing.

@dynamobi-build
Copy link
Author

[author="jvs", created="Sun, 13 Aug 2006 15:55:30 -0500 (GMT-05:00)"]
Discussion in FRG-182 has another example of where controlling the verbosity would be nice.

@dynamobi-build
Copy link
Author

[author="jvs", created="Thu, 20 Sep 2007 21:18:34 -0500 (GMT-05:00)"]
LDB-168 logs the need for better reporting of cost in final plan for LucidDB.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant