eliaperantoni commented on code in PR #13664:
URL: https://github.com/apache/datafusion/pull/13664#discussion_r1932071276
##########
datafusion/sqllogictest/test_files/unnest.slt:
##########
@@ -899,7 +899,7 @@ logical_plan
07)--------Unnest: lists[__unnest_placeholder(outer_ref(u.column1))|depth=1]
structs[]
08)----------Projection: outer_ref(u.column1) AS
__unnest_placeholder(outer_ref(u.column1))
09)------------EmptyRelation
-physical_plan_error This feature is not implemented: Physical plan does not
support logical expression OuterReferenceColumn(List(Field { name: "item",
data_type: Int64, nullable: true, dict_id: 0, dict_is_ordered: false, metadata:
{} }), Column { relation: Some(Bare { table: "u" }), name: "column1" })
+physical_plan_error This feature is not implemented: Physical plan does not
support logical expression OuterReferenceColumn(List(Field { name: "item",
data_type: Int64, nullable: true, dict_id: 0, dict_is_ordered: false, metadata:
{} }), Column { relation: Some(Bare { table: "u" }), name: "column1", spans:
Spans([]) })
Review Comment:
Hmm yeah in general I think I agree that we don't need `Column::Debug` to
print the spans. I'll change it.
--
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: github-unsubscr...@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org
