david-streamlio opened a new issue #9724:
URL: https://github.com/apache/pulsar/issues/9724


   The scope of current implementation of the 
org.apache.pulsar.io.PulsarFunctionE2ETest class is too broad, it is 
responsible for the end to end testing of Pulsar Functions, Pulsar Sources, and 
Pulsar Sinks. These test suites should be implemented in different test classes 
altogether since they test fundamentally different things.
   
   Furthermore, there is a lot of code inside the setup() method that should be 
moved into separate individual methods for ease of maintenance going forward
   
   **Is your enhancement request related to a problem? Please describe.**
   No functional problem per se, just software maintainability
   
   **Describe the solution you'd like**
   Break the PulsarFunctionE2ETest into 4 separate classes: 
PulsarFunctionE2ETest, PulsarBatchE2ETest, PulsarSourceE2ETest, and 
PulsarSinkE2ETest along with a base class that contains common code that is 
required for each Test.
   
   **Describe alternatives you've considered**
   Leaving the code as-is
   
   **Additional context**
   N/A
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to