Copilot commented on code in PR #6523:
URL: https://github.com/apache/ignite-3/pull/6523#discussion_r2314672524
##########
modules/platforms/cpp/ignite/client/ignite_client_configuration.h:
##########
@@ -135,7 +144,7 @@ class ignite_client_configuration {
*
* @return Active connection limit.
*/
- [[nodiscard]] uint32_t get_connection_limit() const { return
m_connection_limit; }
+ [[nodiscard]] std::uint32_t get_connection_limit() const { return
m_connection_limit; }
Review Comment:
[nitpick] The change from `uint32_t` to `std::uint32_t` is inconsistent with
the existing codebase style. If this is intended as a style improvement, it
should be applied consistently throughout the class and codebase.
```suggestion
[[nodiscard]] uint32_t get_connection_limit() const { return
m_connection_limit; }
```
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
