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)

Reply via email to