mirror of https://github.com/apache/kafka.git
Fix typos in multiple files (#19086)
Fix typos in multiple files Reviewers: Andrew Schofield <aschofield@confluent.io>
This commit is contained in:
parent
c1fc59fc23
commit
e4ece37dbf
|
@ -349,7 +349,7 @@ The project maintains the following source code repositories:
|
||||||
Angular JS, v1.6.6
|
Angular JS, v1.6.6
|
||||||
* License MIT (http://www.opensource.org/licenses/mit-license.php)
|
* License MIT (http://www.opensource.org/licenses/mit-license.php)
|
||||||
* Project: http://angularjs.org
|
* Project: http://angularjs.org
|
||||||
* Coyright: (c) 2010-2017 Google, Inc.
|
* Copyright: (c) 2010-2017 Google, Inc.
|
||||||
|
|
||||||
aopalliance Version 1
|
aopalliance Version 1
|
||||||
* License: all the source code provided by AOP Alliance is Public Domain.
|
* License: all the source code provided by AOP Alliance is Public Domain.
|
||||||
|
|
|
@ -135,7 +135,7 @@ public abstract class KafkaFuture<T> implements Future<T> {
|
||||||
* The action may be invoked by the thread that calls {@code whenComplete} or it may be invoked by the thread that
|
* The action may be invoked by the thread that calls {@code whenComplete} or it may be invoked by the thread that
|
||||||
* completes the future.
|
* completes the future.
|
||||||
*
|
*
|
||||||
* @param action the action to preform
|
* @param action the action to perform
|
||||||
* @return the new future
|
* @return the new future
|
||||||
*/
|
*/
|
||||||
public abstract KafkaFuture<T> whenComplete(BiConsumer<? super T, ? super Throwable> action);
|
public abstract KafkaFuture<T> whenComplete(BiConsumer<? super T, ? super Throwable> action);
|
||||||
|
|
|
@ -304,7 +304,7 @@ public class KafkaConsumerTest {
|
||||||
|
|
||||||
KafkaMetric existingMetric = (KafkaMetric) consumer.metrics().entrySet().iterator().next().getValue();
|
KafkaMetric existingMetric = (KafkaMetric) consumer.metrics().entrySet().iterator().next().getValue();
|
||||||
consumer.registerMetricForSubscription(existingMetric);
|
consumer.registerMetricForSubscription(existingMetric);
|
||||||
// This test would fail without the check as the exising metric is registered in the consumer on startup
|
// This test would fail without the check as the existing metric is registered in the consumer on startup
|
||||||
Mockito.verify(clientTelemetryReporter, atMostOnce()).metricChange(existingMetric);
|
Mockito.verify(clientTelemetryReporter, atMostOnce()).metricChange(existingMetric);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -2728,7 +2728,7 @@ public class KafkaProducerTest {
|
||||||
props, new StringSerializer(), new StringSerializer())) {
|
props, new StringSerializer(), new StringSerializer())) {
|
||||||
KafkaMetric existingMetric = (KafkaMetric) producer.metrics().entrySet().iterator().next().getValue();
|
KafkaMetric existingMetric = (KafkaMetric) producer.metrics().entrySet().iterator().next().getValue();
|
||||||
producer.registerMetricForSubscription(existingMetric);
|
producer.registerMetricForSubscription(existingMetric);
|
||||||
// This test would fail without the check as the exising metric is registered in the producer on startup
|
// This test would fail without the check as the existing metric is registered in the producer on startup
|
||||||
Mockito.verify(clientTelemetryReporter, atMostOnce()).metricChange(existingMetric);
|
Mockito.verify(clientTelemetryReporter, atMostOnce()).metricChange(existingMetric);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -2747,7 +2747,7 @@ public class KafkaProducerTest {
|
||||||
props, new StringSerializer(), new StringSerializer())) {
|
props, new StringSerializer(), new StringSerializer())) {
|
||||||
KafkaMetric existingMetric = (KafkaMetric) producer.metrics().entrySet().iterator().next().getValue();
|
KafkaMetric existingMetric = (KafkaMetric) producer.metrics().entrySet().iterator().next().getValue();
|
||||||
producer.unregisterMetricFromSubscription(existingMetric);
|
producer.unregisterMetricFromSubscription(existingMetric);
|
||||||
// This test would fail without the check as the exising metric is registered in the consumer on startup
|
// This test would fail without the check as the existing metric is registered in the consumer on startup
|
||||||
Mockito.verify(clientTelemetryReporter, never()).metricRemoval(existingMetric);
|
Mockito.verify(clientTelemetryReporter, never()).metricRemoval(existingMetric);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -1258,7 +1258,7 @@ public final class KafkaConfigBackingStore extends KafkaTopicBasedBackingStore i
|
||||||
} else {
|
} else {
|
||||||
// TRACE level since there may be many of these records in the config topic
|
// TRACE level since there may be many of these records in the config topic
|
||||||
log.trace(
|
log.trace(
|
||||||
"Ignoring old logging level {} for namespace {} that was writen to the config topic before this worker completed startup",
|
"Ignoring old logging level {} for namespace {} that was written to the config topic before this worker completed startup",
|
||||||
level,
|
level,
|
||||||
namespace
|
namespace
|
||||||
);
|
);
|
||||||
|
|
|
@ -495,7 +495,7 @@ abstract class EmbeddedConnect {
|
||||||
Response response = requestPost(restartEndpoint, "", Collections.emptyMap());
|
Response response = requestPost(restartEndpoint, "", Collections.emptyMap());
|
||||||
try {
|
try {
|
||||||
if (response.getStatus() < Response.Status.BAD_REQUEST.getStatusCode()) {
|
if (response.getStatus() < Response.Status.BAD_REQUEST.getStatusCode()) {
|
||||||
//only the 202 stauts returns a body
|
//only the 202 status returns a body
|
||||||
if (response.getStatus() == Response.Status.ACCEPTED.getStatusCode()) {
|
if (response.getStatus() == Response.Status.ACCEPTED.getStatusCode()) {
|
||||||
return mapper.readerFor(ConnectorStateInfo.class)
|
return mapper.readerFor(ConnectorStateInfo.class)
|
||||||
.readValue(responseToString(response));
|
.readValue(responseToString(response));
|
||||||
|
|
|
@ -493,7 +493,7 @@ public class CoordinatorRuntime<S extends CoordinatorShard<U>, U> implements Aut
|
||||||
final MemoryRecordsBuilder builder;
|
final MemoryRecordsBuilder builder;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* The timer used to enfore the append linger time if
|
* The timer used to enforce the append linger time if
|
||||||
* it is non-zero.
|
* it is non-zero.
|
||||||
*/
|
*/
|
||||||
final Optional<TimerTask> lingerTimeoutTask;
|
final Optional<TimerTask> lingerTimeoutTask;
|
||||||
|
|
|
@ -224,7 +224,7 @@ class CachedPartition(var topic: String,
|
||||||
* fields are read or modified. This includes modification of the session partition map.
|
* fields are read or modified. This includes modification of the session partition map.
|
||||||
*
|
*
|
||||||
* @param id The unique fetch session ID.
|
* @param id The unique fetch session ID.
|
||||||
* @param privileged True if this session is privileged. Sessions crated by followers
|
* @param privileged True if this session is privileged. Sessions created by followers
|
||||||
* are privileged; session created by consumers are not.
|
* are privileged; session created by consumers are not.
|
||||||
* @param partitionMap The CachedPartitionMap.
|
* @param partitionMap The CachedPartitionMap.
|
||||||
* @param usesTopicIds True if this session is using topic IDs
|
* @param usesTopicIds True if this session is using topic IDs
|
||||||
|
|
|
@ -2734,7 +2734,7 @@ public class SharePartitionManagerTest {
|
||||||
|
|
||||||
SharePartition sp1 = mock(SharePartition.class);
|
SharePartition sp1 = mock(SharePartition.class);
|
||||||
// Do not make the share partition acquirable hence it shouldn't be removed from the cache,
|
// Do not make the share partition acquirable hence it shouldn't be removed from the cache,
|
||||||
// as it won't be part of replica manger readFromLog request.
|
// as it won't be part of replica manager readFromLog request.
|
||||||
when(sp1.maybeAcquireFetchLock()).thenReturn(false);
|
when(sp1.maybeAcquireFetchLock()).thenReturn(false);
|
||||||
when(sp1.maybeInitialize()).thenReturn(CompletableFuture.completedFuture(null));
|
when(sp1.maybeInitialize()).thenReturn(CompletableFuture.completedFuture(null));
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue