feat: add static_name() to ExecutionPlan

Open waynexia opened this issue 1 year ago • 2 comments

Which issue does this PR close?

Closes #10246

Rationale for this change

Add static_name() method to make the name of an execution plan accessible without constructing the struct.

What changes are included in this PR?

A new public API in ExecutionPlan

Are these changes tested?

Yes

Are there any user-facing changes?

Yes

Apr 27 '24 14:04 waynexia

@waynexia This looks good, but could you add a test to demonstrate what the output looks like?

Apr 28 '24 00:04 andygrove

@waynexia This looks good, but could you add a test to demonstrate what the output looks like?

Thanks for reviewing. I added one in https://github.com/apache/datafusion/pull/10266/commits/fbb5f4acf7ce3c936c9af783aad0e19328c2e1aa, please check it out

Apr 28 '24 03:04 waynexia

This is nice @waynexia

Apr 29 '24 12:04 matthewmturner