Sergey Chernov
Sergey Chernov
The main issue here is that API do not provide a good way to access queryId after execution. Also generating query may be useful while tracing - so we need...
`InsertResponse` and `QueryResponse` classes have `#getQueryId()` method. However not every operation returns it. In case of failure it should be propagated to the logging. Client does retries. Should queryId be...
Good day, @patschl ! The query ID is generated on server is missing in the request settings. This issue is more about adding query ID whenever it is useful while...
If query reached server then we will get query id in header and will provide in response object. Else query id doesn't make sense because it is not present on...
no worries, @patschl ! Thank you for pointing out - I was not aware of this. I will review what the project does and see if I can help here....
Good day, @Onajk ! I think so, I will look into the problem.
@al-tr this is related to the https://github.com/ClickHouse/clickhouse-java/issues/2389 We will fix in nearest future.
@copilot great work! Please sign CLA. Thanks!
@copilot there seems 0% coverage of the new code. Please check.
@copilot some checks failed: ``` Error: Failures: Error: ClientTests.testDefaultSettings:224 expected [32] but found [33] Error: ClientTests.testWithOldDefaults:324 expected [32] but found [33] ```