ruanwenjun commented on a change in pull request #1536:
URL:
https://github.com/apache/incubator-seatunnel/pull/1536#discussion_r832878183
##########
File path: pom.xml
##########
@@ -80,6 +80,7 @@
<module>seatunnel-connectors</module>
<module>seatunnel-dist</module>
<module>seatunnel-examples</module>
+ <module>seatunnel-e2e</module>
Review comment:
@CalvinKirs Thanks for your review, I got you. Do we also need to split
e2e with integration test? In some projects, they only have IT and some have
both e2e, IT. If we also need IT, I think put IT in the `seatunnel` project is
OK. Maybe this PR is more like to only add IT.
Agree with use of another CI for e2e.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
