apache / apache/kyuubi

[Bug] Throw the exception to notify FlinkEngine config `table.dml-sync` is not equal to the parameter runAsync

Open
#2,166 6 comments 0 reactions 1 assignee Claimed by @fsk119 View on GitHub
kind:bug priority:major
Dominant language
Scala
Stars
2.4k
Forks
1k
PR merge metrics
No merged PRs in 30d

Description

### Code of Conduct

- [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)

### Search before asking

- [x] I have searched in the [issues](https://github.com/apache/incubator-kyuubi/issues?q=is%3Aissue) and found no similar issues.

### Describe the bug

Flink table uses `table.dml-sync`[1] to control whether wait the execution finish. However, Kyuubi use its own parameter to control the behaviour. I think it's neccesary to keep the behaviour unified and throw the exception to notify the users.

[1] https://nightlies.apache.org/flink/flink-docs-master/docs/dev/table/config/#table-dml-sync

### Affects Version(s)

master

### Kyuubi Server Log Output

_No response_

### Kyuubi Engine Log Output

_No response_

### Kyuubi Server Configurations

_No response_

### Kyuubi Engine Configurations

_No response_

### Additional context

_No response_

### Are you willing to submit PR?

- [X] Yes I am willing to submit a PR!

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.