Skip to content

Commit 36d2498

Browse files
MINOR: Use meaningful name in AsyncKafkaConsumerTest (#19550)
Replace names like a, b, c, ... with meaningful names in AsyncKafkaConsumerTest. Follow-up: #19457 (comment) Signed-off-by: PoAn Yang <[email protected]> Reviewers: Bill Bejeck <[email protected]>, Ken Huang <[email protected]>
1 parent 22b89b6 commit 36d2498

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

clients/src/test/java/org/apache/kafka/clients/consumer/internals/AsyncKafkaConsumerTest.java

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -223,10 +223,10 @@ private AsyncKafkaConsumer<String, String> newConsumerWithStreamRebalanceData(
223223
new StringDeserializer(),
224224
new StringDeserializer(),
225225
time,
226-
(a, b, c, d, e, f, g, h) -> applicationEventHandler,
227-
a -> backgroundEventReaper,
228-
(a, b, c, d, e, f, g) -> fetchCollector,
229-
(a, b, c, d) -> metadata,
226+
(logContext, time, applicationEventBlockingQueue, completableEventReaper, applicationEventProcessorSupplier, networkClientDelegateSupplier, requestManagersSupplier, asyncConsumerMetrics) -> applicationEventHandler,
227+
logContext -> backgroundEventReaper,
228+
(logContext, consumerMetadata, subscriptionState, fetchConfig, deserializers, fetchMetricsManager, time) -> fetchCollector,
229+
(consumerConfig, subscriptionState, logContext, clusterResourceListeners) -> metadata,
230230
backgroundEventQueue,
231231
Optional.ofNullable(streamsRebalanceData)
232232
);
@@ -238,10 +238,10 @@ private AsyncKafkaConsumer<String, String> newConsumer(ConsumerConfig config) {
238238
new StringDeserializer(),
239239
new StringDeserializer(),
240240
time,
241-
(a, b, c, d, e, f, g, h) -> applicationEventHandler,
242-
a -> backgroundEventReaper,
243-
(a, b, c, d, e, f, g) -> fetchCollector,
244-
(a, b, c, d) -> metadata,
241+
(logContext, time, applicationEventBlockingQueue, completableEventReaper, applicationEventProcessorSupplier, networkClientDelegateSupplier, requestManagersSupplier, asyncConsumerMetrics) -> applicationEventHandler,
242+
logContext -> backgroundEventReaper,
243+
(logContext, consumerMetadata, subscriptionState, fetchConfig, deserializers, fetchMetricsManager, time) -> fetchCollector,
244+
(consumerConfig, subscriptionState, logContext, clusterResourceListeners) -> metadata,
245245
backgroundEventQueue,
246246
Optional.empty()
247247
);

0 commit comments

Comments
 (0)