Hi @Jing Ge Thanks for the proposal and sorry for joining the discussion late. I met the same problem when I performed a batch query on sql-client and it could not print the whole line of results.
I think the FLIP is nice, just one comment: Can users set the parameter dynamically on sql-client with command 'set table.display.max-column-width=100'? If it can, it will be more convenient for users to use sql-client. Best, Shammon On Mon, Jan 9, 2023 at 1:04 PM Jing Ge <j...@ververica.com.invalid> wrote: > Hi Dev, > > I let the discussion open for a longer time because of the holiday session. > It seems there are no concerns about this proposal. I will start a voting > thread. Thanks! > > Best regards, > Jing > > On Fri, Dec 16, 2022 at 11:09 AM Jing Ge <j...@ververica.com> wrote: > > > Hi Dev, > > > > I'd like to start a discussion about FLIP-279: Unified the max display > > column width for SqlClient and Table APi in both Streaming and Batch > > execMode[1]. > > > > On one side, currently, there is only one ConfigOption > > > SqlClientOptions.DISPLAY_MAX_COLUMN_WIDTH(sql-client.display.max-column-width') > > could be used to configure the value and it only works in a very specific > > case. Users need to truly understand it before using it correctly. > > > > On the other side, users want to configure the max display column width > > while: > > > > - using CLI > > - Using Table API > > - Running job in batch execution mode > > - Running job in streaming execution mode > > > > > > FLIP-279 will try to provide a unified way to address this issue. Looking > > forward to your feedback. > > > > Best regards, > > Jing > > > > [1] > > > https://cwiki.apache.org/confluence/display/FLINK/FLIP-279+Unified+the+max+display+column+width+for+SqlClient+and+Table+APi+in+both+Streaming+and+Batch+execMode > > > > >
