You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@flink.apache.org by "Matthias Pohl (Jira)" <ji...@apache.org> on 2023/02/10 10:27:00 UTC

[jira] [Created] (FLINK-31014) Expose exceptions in connector testing framework

Matthias Pohl created FLINK-31014:
-------------------------------------

             Summary: Expose exceptions in connector testing framework
                 Key: FLINK-31014
                 URL: https://issues.apache.org/jira/browse/FLINK-31014
             Project: Flink
          Issue Type: Improvement
          Components: API / DataStream, Test Infrastructure
    Affects Versions: 1.16.1, 1.17.0, 1.18.0
            Reporter: Matthias Pohl


I noticed for a few interfaces that we don't expose checked exceptions where it's actually useful. {{DataStreamSinkV2ExternalContext.createSink}}, for instance, is not exposing any Throwable except for {{UnsupportedOperationException}}. This implies that internal caught exception need to be either caught and transformed into a {{RuntimeException}} or handled internally. AFAIU, that's not what we want. The connector testing framework wants exceptions to be exposed. Therefore, we should improve the interfaces of the framework to allow exceptions to happen (probably depending on the usecase).

This makes it easier for contributors implementing test cases for their connector giving them the freedom to just expose undesired exceptions instead of handling them internally. JUnit should be the actor dealing with these exceptions.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)