Closed jonahgao closed 1 year ago
None
There is a duplicate invocation of create_logical_plan within ctx.sql(sql). Since we already have a logical plan at hand, perhaps we can execute it directly.
create_logical_plan
ctx.sql(sql)
No
Good catch. Thanks @jonahgao.
yahoNanJing
commented
1 year ago
Which issue does this PR close?
None
Rationale for this change
There is a duplicate invocation of
create_logical_planwithinctx.sql(sql). Since we already have a logical plan at hand, perhaps we can execute it directly.What changes are included in this PR?
Are there any user-facing changes?
No