Skip to content

Commit 7b29916

Browse files
committed
spring-projectsGH-2154: Fix Tombestones with Delegating Serial...
Resolves spring-projects#2154 Check for null and return.
1 parent bbf26bc commit 7b29916

File tree

3 files changed

+12
-0
lines changed

3 files changed

+12
-0
lines changed

Diff for: spring-kafka/src/main/java/org/springframework/kafka/support/serializer/DelegatingByTopicSerializer.java

+3
Original file line numberDiff line numberDiff line change
@@ -74,6 +74,9 @@ public byte[] serialize(String topic, Object data) {
7474
@SuppressWarnings("unchecked")
7575
@Override
7676
public byte[] serialize(String topic, Headers headers, Object data) {
77+
if (data == null) {
78+
return null;
79+
}
7780
return ((Serializer<Object>) findDelegate(topic)).serialize(topic, headers, data);
7881
}
7982

Diff for: spring-kafka/src/main/java/org/springframework/kafka/support/serializer/DelegatingByTypeSerializer.java

+6
Original file line numberDiff line numberDiff line change
@@ -83,12 +83,18 @@ public void configure(Map<String, ?> configs, boolean isKey) {
8383

8484
@Override
8585
public byte[] serialize(String topic, Object data) {
86+
if (data == null) {
87+
return null;
88+
}
8689
Serializer<Object> delegate = findDelegate(data, this.delegates);
8790
return delegate.serialize(topic, data);
8891
}
8992

9093
@Override
9194
public byte[] serialize(String topic, Headers headers, Object data) {
95+
if (data == null) {
96+
return null;
97+
}
9298
Serializer<Object> delegate = findDelegate(data, this.delegates);
9399
return delegate.serialize(topic, headers, data);
94100
}

Diff for: spring-kafka/src/main/java/org/springframework/kafka/support/serializer/DelegatingSerializer.java

+3
Original file line numberDiff line numberDiff line change
@@ -190,6 +190,9 @@ public byte[] serialize(String topic, Object data) {
190190

191191
@Override
192192
public byte[] serialize(String topic, Headers headers, Object data) {
193+
if (data == null) {
194+
return null;
195+
}
193196
byte[] value = null;
194197
String selectorKey = selectorKey();
195198
Header header = headers.lastHeader(selectorKey);

0 commit comments

Comments
 (0)