link3280 commented on issue #4083: URL: https://github.com/apache/kyuubi/issues/4083#issuecomment-1370756561
@bobo495 Thanks for reporting the problem. I agree that the current implementation of the result retrieval of the Flink engine is improbable. The final solution would refactor it to return results in a streaming manner, which is mentioned before but the related work doesn't start yet. WRT you suggestions > When snapshotResult gets the result, jumps out of the loop. I think the current behavior is expected. `snapshotResult` returns paged returns, and we fetch them page by page. > Loop sets the maximum timeout to avoid being unable to exit the loop if the result is null. It's a viable approach in addition to max result rows for now. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
