xuefuz commented on issue #8727: [FLINK-12828][sql-client] Support -f option with a sql script file as… URL: https://github.com/apache/flink/pull/8727#issuecomment-526298331 @docete It shouldn't matter if a command spans multiple line or a line contains multiple commands in the script file, one should be able to read the file content and parse out one command at a time to execute. This avoids holding the file content in memory. Nevertheless, I guess the current way is acceptable for now. We can worry about memory usage later. Thanks again for working on this.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
