[Java][Debezium] Fix NPE in debeziumRecordInstant for DELETE events

sdks/java/io/debezium/src/main/java/org/apache/beam/io/debezium/KafkaConnectUtils.java

@@ -80,15 +80,26 @@ public static Schema.FieldType beamSchemaTypeFromKafkaType(
   }
 
   public static Instant debeziumRecordInstant(SourceRecord record) {
-    if (!record.valueSchema().type().equals(org.apache.kafka.connect.data.Schema.Type.STRUCT)
-        || record.valueSchema().field("ts_ms") == null) {
-      throw new IllegalArgumentException(
-          "Debezium record received is not of the right kind. "
-              + String.format(
-                  "Should be STRUCT with ts_ms field. Instead it is: %s", record.valueSchema()));
+    if (record.valueSchema() != null
+        && record.valueSchema().type().equals(org.apache.kafka.connect.data.Schema.Type.STRUCT)
+        && record.valueSchema().field("ts_ms") != null
+        && record.value() != null) {
+      Struct recordValue = (Struct) record.value();
+      return Instant.ofEpochMilli(recordValue.getInt64("ts_ms"));
     }
-    Struct recordValue = (Struct) record.value();
-    return Instant.ofEpochMilli(recordValue.getInt64("ts_ms"));
+
+    if (record.sourceOffset() != null && record.sourceOffset().containsKey("ts_usec")) {
+      Object tsUsecValue = record.sourceOffset().get("ts_usec");
+      if (tsUsecValue instanceof Number) {
+        return Instant.ofEpochMilli(((Number) tsUsecValue).longValue() / 1000);
+      }
+    }
+
+    throw new IllegalArgumentException(
+        "Debezium record received is not of the right kind. "
+            + String.format(
+                "Should be STRUCT with ts_ms field or sourceOffset with ts_usec. Instead it is: %s, %s",
+                record.valueSchema(), record.sourceOffset()));
   }
 
   public static SourceRecordMapper<Row> beamRowFromSourceRecordFn(final Schema recordSchema) {

sdks/java/io/debezium/src/test/java/org/apache/beam/io/debezium/KafkaConnectSchemaTest.java

@@ -64,6 +64,40 @@ public void testTimestampRequired() {
     IllegalArgumentException e =
         assertThrows(
             IllegalArgumentException.class, () -> KafkaConnectUtils.debeziumRecordInstant(record));
-    assertThat(e.getMessage(), Matchers.containsString("Should be STRUCT with ts_ms field"));
+    assertThat(
+        e.getMessage(),
+        Matchers.containsString("Should be STRUCT with ts_ms field or sourceOffset with ts_usec"));
+  }
+
+  @Test
+  public void testDebeziumRecordInstantNullValueSchema() {
+    org.apache.kafka.connect.source.SourceRecord record =
+        new org.apache.kafka.connect.source.SourceRecord(
+            java.util.Collections.singletonMap("server", "test"),
+            java.util.Collections.singletonMap("ts_usec", 1614854400000000L),
+            "test-topic",
+            null,
+            null);
+
+    org.joda.time.Instant instant = KafkaConnectUtils.debeziumRecordInstant(record);
+    assertThat(instant.getMillis(), Matchers.is(1614854400000L));
+  }
+
+  @Test
+  public void testDebeziumRecordInstantMissingTimestamp() {
+    org.apache.kafka.connect.source.SourceRecord record =
+        new org.apache.kafka.connect.source.SourceRecord(
+            java.util.Collections.singletonMap("server", "test"),
+            java.util.Collections.emptyMap(),
+            "test-topic",
+            null,
+            null);
+
+    IllegalArgumentException e =
+        assertThrows(
+            IllegalArgumentException.class, () -> KafkaConnectUtils.debeziumRecordInstant(record));
+    assertThat(
+        e.getMessage(),
+        Matchers.containsString("Should be STRUCT with ts_ms field or sourceOffset with ts_usec"));
   }
 }