getRetainedSizeInBytes implements method in ConnectorSplit; expected @OverRide

Importing nested classes/static methods/static fields with commonly-used names can make code harder to read, because it may not be clear from the context exactly which type is being referred to. Qualifying the name with that of the containing class can make the code clearer. Here we recommend using qualified class: StorageTransactionHandle.

Using stream::iterator creates a one-shot Iterable, which may cause surprising failures.

It is very rare for LinkedList to out-perform ArrayList or ArrayDeque. Avoid it unless you're willing to invest a lot of time into benchmarking. Caveat: LinkedList supports null elements, but ArrayDeque does not.

Implicit use of the platform default charset, which can result in differing behaviour between JVM executions or incorrect behavior if the encoding of the data source doesn't match expectations.

Implicit use of the platform default charset, which can result in differing behaviour between JVM executions or incorrect behavior if the encoding of the data source doesn't match expectations.

This code can be simplified to use a pattern-matching instanceof.

This code can be simplified to use a pattern-matching instanceof.

This code can be simplified to use a pattern-matching instanceof.

Caught exceptions should not be ignored