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
| 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
| 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
| 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. |
3 participants
#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