Copilot commented on code in PR #6645:
URL: https://github.com/apache/ignite-3/pull/6645#discussion_r2374968913
##########
modules/client/src/test/java/org/apache/ignite/client/RetryPolicyTest.java:
##########
@@ -242,7 +242,7 @@ public void
testRetryPolicyConvertOpAllOperationsSupported() throws IllegalAcces
@Test
public void testRetryReadPolicyAllOperationsSupported() {
var plc = new RetryReadPolicy();
- var cfg = new IgniteClientConfigurationImpl(null, null, 0, 0, null, 0,
0, null, null, null, false, null, 0, 1024);
+ var cfg = new IgniteClientConfigurationImpl(null, null, 0, 0, null, 0,
0, null, null, null, false, null, 0, 1024, null);
Review Comment:
[nitpick] This constructor call with many null parameters is hard to read
and maintain. Consider using a builder pattern or factory method for test
configuration creation.
--
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]
