[GitHub] [incubator-kyuubi] fsk119 commented on issue #2166: [Bug] Throw the exception to notify FlinkEngine config `table.dml-sync` is not equal to the parameter runAsync

[GitBox <gi...@apache.org>]

fsk119 commented on issue #2166:
URL: https://github.com/apache/incubator-kyuubi/issues/2166#issuecomment-1108215020

   Hi, all. Sorry for the late response. I think it's not a bug after reconsider the issue. The option `table.dml-sync` means the Flink `TableEnvironment` will wait for the execution finish, which means the lifecycle of the `Operation` contains the job execution. But the sync mode in the gateway means the Gateway should return the signal to the client until the `Operation` finishes, which doesn't means the gateway should influence the lifecycle of the `Operation`. Therefore, I think the two options don't need be the same. WDYT?
   
   But the Operation lifecycle in the Hive always contains the execution of the job[1], right? Maybe we should set the option true forcely? 
   
   [1]https://github.com/apache/hive/blob/23b0d30051af3ade5b03496ab9cf22929168dc84/service/src/java/org/apache/hive/service/cli/operation/SQLOperation.java#L233
   


-- 
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: notifications-unsubscribe@kyuubi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org