slinkydeveloper commented on a change in pull request #17739:
URL: https://github.com/apache/flink/pull/17739#discussion_r748291254
##########
File path:
flink-table/flink-table-api-java/src/main/java/org/apache/flink/table/api/internal/TableResultImpl.java
##########
@@ -248,96 +223,13 @@ public Builder setPrintStyle(PrintStyle printStyle) {
return this;
}
- /** Specifies session time zone. */
- public Builder setSessionTimeZone(ZoneId sessionTimeZone) {
- Preconditions.checkNotNull(sessionTimeZone, "sessionTimeZone
should not be null");
- this.sessionTimeZone = sessionTimeZone;
- return this;
- }
-
/** Returns a {@link TableResult} instance. */
public TableResultInternal build() {
+ if (printStyle == null) {
+ printStyle =
PrintStyle.rawContent(resultProvider.getRowDataStringConverter());
Review comment:
The default it's the raw content style, but now it needs the
`RowDataStringConverter` from `ResultProvider`, that's why I had to move it
here.
--
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]
