Copilot commented on code in PR #17337:
URL: https://github.com/apache/datafusion/pull/17337#discussion_r2429395856
##########
datafusion/core/tests/user_defined/user_defined_plan.rs:
##########
@@ -262,8 +262,9 @@ async fn topk_query() -> Result<()> {
async fn topk_plan() -> Result<()> {
let ctx = setup_table(make_topk_context()).await?;
+ #[rustfmt::skip]
let mut expected = ["| logical_plan after topk
| TopK: k=3
|",
- "| |
TableScan: sales projection=[customer_id,revenue]
|"].join("\n");
+ "| |
TableScan:salesprojection=[customer_id,revenue]preferred_ordering=[sales.revenueDESCNULLSFIRST]
|"].join("\n");
Review Comment:
Missing spaces in the expected test output string. Should have spaces around
'sales', 'projection=', 'preferred_ordering=', 'revenue', 'DESC', 'NULLS', and
'FIRST'.
```suggestion
"| |
TableScan: sales projection=[customer_id, revenue]
preferred_ordering=[sales.revenue DESC NULLS FIRST]
|"].join("\n");
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
