Hi everyone,

pretty much every connector utilizes checks from the
org.apache.flink.util.Preconditions class, which is currently marked
as @Internal. This causes CONNECTOR_CLASSES_ONLY_DEPEND_ON_PUBLIC_API
architecture tests to fail and results in the violation exceptions being
filled with the lines like this:
https://github.com/apache/flink-connector-kafka/blob/8d531898d7df775535084e0413442c86a4c497ed/flink-connector-kafka/archunit-violations/984f05c0-ec82-405e-9bcc-d202dbe7202e#L22-L27

I'd like to propose to mark the Preconditions class as @Public of
@PublicEvolving. There are also some other similar candidates, like
ClosureCleaner, but the Preconditions is by far the most widespread.
Alternatively we could add an exception to the ArchRule definition.

What do you think?

Best,
Alex

Reply via email to