[jira] [Created] (FLINK-22277) Refactor sql client's DynamicResult

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Created] (FLINK-22277) Refactor sql client's DynamicResult

Shang Yuanchun (Jira)
Kurt Young created FLINK-22277:
----------------------------------

             Summary: 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


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)