[GitHub] dawidwys commented on issue #6506: [FLINK-10073] [sql-client] Allow setting a restart strategy in SQL Client

2018-08-07 Thread GitBox
dawidwys commented on issue #6506: [FLINK-10073] [sql-client] Allow setting a 
restart strategy in SQL Client
URL: https://github.com/apache/flink/pull/6506#issuecomment-411089108
 
 
   You are right it is not supported in the DataStream API, but is supported 
from flink-conf.yaml via reflection (which is a bit weird). Anyway I was in 
favor of **not** adding such possibility (unfortunately haven't expressed it 
clearly). Still  +1 from side.


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] dawidwys commented on issue #6506: [FLINK-10073] [sql-client] Allow setting a restart strategy in SQL Client

2018-08-07 Thread GitBox
dawidwys commented on issue #6506: [FLINK-10073] [sql-client] Allow setting a 
restart strategy in SQL Client
URL: https://github.com/apache/flink/pull/6506#issuecomment-411080017
 
 
   @twalthr Do you think we should also support user defined restart 
strategies? If not, which truth be told I would be in favor, then I am +1.


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:
us...@infra.apache.org


With regards,
Apache Git Services