Add new user doc to translate logical plan to physical plan #12026
Conversation
github-actions
bot
added
the
documentation
docs/src/library_logical_plan.rs
Outdated
| @@ -76,3 +79,32 @@ fn plan_builder_1() -> Result<()> { | |||
|
|
|||
| Ok(()) | |||
| } | |||
|
|
|||
| #[tokio::test] | |||
There was a problem hiding this comment.
This file is confusing -- i don't think it is used anymore --- I added #12081 to remove it
| @@ -131,6 +131,37 @@ Filter: person.id > Int32(500) [id:Int32;N, name:Utf8;N] | |||
| TableScan: person [id:Int32;N, name:Utf8;N] | |||
| ``` | |||
|
|
|||
| ## Translating Logical Plan to Physical Plan | |||
There was a problem hiding this comment.
Thanks @jc4x4 -- I have some ideas on improving this example -- I will push some suggestions to this branch
| @@ -678,6 +678,12 @@ doc_comment::doctest!( | |||
| library_user_guide_sql_api | |||
| ); | |||
|
|
|||
| #[cfg(doctest)] | |||
There was a problem hiding this comment.
I added this block which means the examples in the library guide are now run as part of cargo test --doc
|
|
||
| // print the plan | ||
| println!("{}", plan.display_indent_schema()); | ||
| use datafusion::common::DataFusionError; |
There was a problem hiding this comment.
I updated these examples so they compile as standalone examples
github-actions
bot
added
logical-expr
I dont mind at all. Thanks for improving it. Let me know if I need to do anything on my end. |
#7306
Which issue does this PR close?
Closes #7306.
Rationale for this change
New user doc for translating logical plan to physical plan
What changes are included in this PR?
create_physical_planAre these changes tested?
Run test and pasted output into md.
Are there any user-facing changes?
Only doc