Commit 5c695348 authored by Stephane Nicoll's avatar Stephane Nicoll

Remove direct dependency on Kafka's IsolationLevel

Closes gh-20811
parent f4e6d7c1
...@@ -30,7 +30,6 @@ import org.apache.kafka.clients.CommonClientConfigs; ...@@ -30,7 +30,6 @@ import org.apache.kafka.clients.CommonClientConfigs;
import org.apache.kafka.clients.consumer.ConsumerConfig; import org.apache.kafka.clients.consumer.ConsumerConfig;
import org.apache.kafka.clients.producer.ProducerConfig; import org.apache.kafka.clients.producer.ProducerConfig;
import org.apache.kafka.common.config.SslConfigs; import org.apache.kafka.common.config.SslConfigs;
import org.apache.kafka.common.requests.IsolationLevel;
import org.apache.kafka.common.serialization.StringDeserializer; import org.apache.kafka.common.serialization.StringDeserializer;
import org.apache.kafka.common.serialization.StringSerializer; import org.apache.kafka.common.serialization.StringSerializer;
...@@ -1222,6 +1221,31 @@ public class KafkaProperties { ...@@ -1222,6 +1221,31 @@ public class KafkaProperties {
} }
public enum IsolationLevel {
/**
* Read everything including aborted transactions.
*/
READ_UNCOMMITTED((byte) 0),
/**
* Read records from committed transactions, in addition to records not part of
* transactions.
*/
READ_COMMITTED((byte) 1);
private final byte id;
IsolationLevel(byte id) {
this.id = id;
}
public byte id() {
return this.id;
}
}
@SuppressWarnings("serial") @SuppressWarnings("serial")
private static class Properties extends HashMap<String, Object> { private static class Properties extends HashMap<String, Object> {
......
/*
* Copyright 2012-2020 the original author or authors.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* https://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package org.springframework.boot.autoconfigure.kafka;
import org.junit.jupiter.api.Test;
import org.springframework.boot.autoconfigure.kafka.KafkaProperties.IsolationLevel;
import static org.assertj.core.api.Assertions.assertThat;
/**
* Tests for {@link KafkaProperties}.
*
* @author Stephane Nicoll
*/
class KafkaPropertiesTests {
@Test
void isolationLevelEnumConsistentWithKafkaVersion() {
org.apache.kafka.common.requests.IsolationLevel[] original = org.apache.kafka.common.requests.IsolationLevel
.values();
assertThat(original).extracting("name").containsExactly(IsolationLevel.READ_UNCOMMITTED.name(),
IsolationLevel.READ_COMMITTED.name());
assertThat(original).extracting("id").containsExactly(IsolationLevel.READ_UNCOMMITTED.id(),
IsolationLevel.READ_COMMITTED.id());
assertThat(original).hasSize(IsolationLevel.values().length);
}
}
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment