-
Notifications
You must be signed in to change notification settings - Fork 559
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pre 0.6.4 Updates #1773
Pre 0.6.4 Updates #1773
Conversation
## 0.6.4 | ||
|
||
### Important | ||
- Following components will be deprecated and archived in next release: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have we confirmed the removal in the next release? If no, I'd move the deprecation warning at the top to ensure it's visible to everyone reading the file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Moved to deprecations section.
- Following components will be deprecated and archived in next release: | ||
- clickhouse-cli-client | ||
- clickhouse-grpc-client | ||
- No more builds for non-lts Java versions - no more Java 9 release builds. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't it a breaking change for users with the old Java version? If so, should we tag the release as 0.7.0
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Discussed. It will be in deprecations warning and will be removed in 0.7.0 release.
CHANGELOG.md
Outdated
(https://github.com/ClickHouse/clickhouse-java/pull/1771) | ||
- com.clickhouse.client.config.ClickHouseClientOption#CONNECTION_TTL - to configure connection time-to-live | ||
- com.clickhouse.client.http.config.ClickHouseHttpOption#KEEP_ALIVE_TIMEOUT - to configure keep-alive timeout | ||
- com.clickhouse.client.http.config.ClickHouseHttpOption#CONNECTION_REUSE_STRATEGY - to configure how connection |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The setting name is self-descriptive. We'd rather focus on the meaning of the setting and possible configuration value. You can even use the existing comments from javadoc
Lines 93 to 96 in 82f8b21
* Configuration for AsyncHttpClient connection pool. It defines how to reuse connections. | |
* If {@code "FIFO"} is set, the connections are reused in the order they were created. | |
* If {@code "LIFO"} is set, the connections are reused as soon they are available. | |
* Default value is {@code "LIFO"}. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated.
CHANGELOG.md
Outdated
Select FIFO to reuse connections evenly or LIFO (default) to reuse the most recently active connections. | ||
- [Client-V2] All operations are now executed in calling thread to avoid extra threads creation. | ||
Async operations can be enabled by `com.clickhouse.client.api.Client.Builder.useAsyncRequests` (https://github.com/ClickHouse/clickhouse-java/pull/1767) | ||
- [Client-V2] Spring Demo Service as usage example (https://github.com/ClickHouse/clickhouse-java/pull/1765) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we group the code and docs changes into relevant sections? it would help to skim through the file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Moved to documentation section.
Quality Gate passedIssues Measures |
Summary