flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (FLINK-8686) Improve basic embedded SQL client
Date Mon, 27 Aug 2018 15:39:00 GMT

    [ https://issues.apache.org/jira/browse/FLINK-8686?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16593836#comment-16593836

ASF GitHub Bot commented on FLINK-8686:

fhueske commented on a change in pull request #6621:  [FLINK-8686] [sql-client] Limit result
size for prototyping modes
URL: https://github.com/apache/flink/pull/6621#discussion_r213010987

 File path: flink-libraries/flink-sql-client/src/main/java/org/apache/flink/table/client/cli/CliChangelogResultView.java
 @@ -133,6 +134,13 @@ protected void refresh() {
 					// update results
+					// formatting and printing of rows is expensive in the current implementation,
+					// therefore we limit the maximum number of lines shown in changelog mode to
+					// keep the CLI responsive
+					if (results.size() >= DEFAULT_MAX_ROW_COUNT) {
+						results.remove(0);
 Review comment:
   Since `results` is an `ArrayList`, removing the first element means that the whole array
is shifted by copying. `LinkedList` might be more efficient. 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:

> Improve basic embedded SQL client 
> ----------------------------------
>                 Key: FLINK-8686
>                 URL: https://issues.apache.org/jira/browse/FLINK-8686
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Table API &amp; SQL
>            Reporter: Timo Walther
>            Assignee: Timo Walther
>            Priority: Major
>              Labels: pull-request-available
> This issue describes follow-up issues that should be fixes in order to make the SQL client
more stable:
>  - -Add more tests for executor-
>  - Configure JVM heap size
>  - Limit changelog and table buffers
>  - -"The input is invalid please check it again." => add allowed range-
>  - Load dependencies recursively
>  - Clean up results in result store
>  - -Improve error message for unsupported batch queries-
>  - -Add more logging instead swallowing exceptions-
>  - -List properties in error message about missing TS factory sorted by name-
>  - Add command to show loaded TS factories and their required propeties
>  - Add command to reload configuration from files (no need to restart client)
>  - Improve error message in case of invalid json-schema (right now: {{java.lang.IllegalArgumentException:
No type could be found in node:<root>}}
>  - -Add switch to show full stacktraces of exceptions- solved by logging
>  - Give error message when setting unknown parameters {{result-mode=changelog}} does
not give an error but should be {{execution.result-mode=changelog}}

This message was sent by Atlassian JIRA

View raw message