Github user walterddr commented on the issue:
https://github.com/apache/flink/pull/5867
Hi @twalthr, having JVM heap size configurable is definitely a great
benefit. Just to clarify, this is only changing the Client JVM heap size,
correct? I am assuming this is mainly for manipulating large size data when
retrieving results from the query.
I think adding it to `flink-conf.yaml` should be fine. There are already
specific configurations commented out for components like Kerberos, ZK, etc.
which makes adding a specific conf to sql-client less disturbing. We can always
break it out to another file later when extra configuration files are needed
for sql-client in the future.
What do you guys think?
---
