[
https://issues.apache.org/jira/browse/FLINK-22277?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17324698#comment-17324698
]
Nicholas Jiang commented on FLINK-22277:
----------------------------------------
[~jark], could you please assign this to me? I'm working for this issue.
> Refactor sql client's DynamicResult
> ------------------------------------
>
> Key: FLINK-22277
> URL: https://issues.apache.org/jira/browse/FLINK-22277
> Project: Flink
> Issue Type: Improvement
> Components: Table SQL / Client
> Reporter: Kurt Young
> Priority: Minor
>
> Right now, the sql client Executor has different result handling logic for
> different result display mode.
> Different result is handled by child classes of `DynamicResult`.
> This leads to introducing page related API to the Executor, such as
> `snapshotResult` and `retrieveResultPage`, which I think is inappropriate
> and will make things complicated.
> It will be benefit to simplify the responsibility of Executor about
> retrieving results to simply streaming back the SELECT result, and move the
> logic of dealing with different display mode into CliResultView.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
