lincoln-lil commented on code in PR #21683:
URL: https://github.com/apache/flink/pull/21683#discussion_r1080828555


##########
flink-table/flink-table-planner/src/main/scala/org/apache/flink/table/planner/plan/utils/FlinkRelOptUtil.scala:
##########
@@ -90,6 +90,58 @@ object FlinkRelOptUtil {
     sw.toString
   }
 
+  /**
+   * Converts a sequence of relational expressions to a string. This is 
different from
+   * [[RelOptUtil]]#toString and overloaded [[FlinkRelOptUtil]]#toString on 
following points:
+   *   - Generated string by this method is in a tree style
+   *   - Generated string by this method may have more information about 
RelNode, such as

Review Comment:
   nit: -> 'Generated string by this method may have available [[PlanAdvice]]'? 
   'retractionTraits' is not easy to understand for developers and I didn't see 
more 'withXX' args enabled  by default.



-- 
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]

Reply via email to