liunaijie commented on PR #5702:
URL: https://github.com/apache/seatunnel/pull/5702#issuecomment-1778852374

   > Could you add a test case for this? Seem like this is a very important 
feature we shouldn't missed. It's weird we don't have a test case to cover it. 
cc @liugddx
   
   reuse the `RestApiIT`, but change something.
   1. remove `Common.setDeployMode` before startup
   when start the st cluster with `./bin/seatunnel-cluster.sh -d` command, the 
deployMode will be null, so delete this setup to keep consistency
   
   2. remove parameter as they are optional when submit job.
   when submit job i think they all should be null. when re-store job they need 
have value.
   
   
   
   
   
   
   
   
   
   


-- 
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]

Reply via email to