Closed huanyu-li closed 3 months ago
I will merge this PR now. The next task then is to extend the new TextBasedLogicalPlanPrinterImpl2
class that you introduce in this PR, by adding the functionality to print logical plans as shown in the mockup.
For that implementation you should not anymore use the toString()
functions of the operators.
The last three commits are for addressing above comments.