zhangshenghang commented on PR #10033:
URL: https://github.com/apache/seatunnel/pull/10033#issuecomment-3521636057

   > > thanks @LiJie20190102 This is likely a problem. Can you verify your 
modifications through e2e?
   > 
   > I don't think e2e is necessary because for Spark, when submitting a task 
as '--master yarn --deploy-mode client', all relevant JARs can be loaded on the 
client side, but for '--master yarn --deploy-mode cluster', some JARs need to 
display declaration loading. At the same time, this requires the yarn 
environment, and I don't know if e2e can complete this function. Finally, I 
compiled it and packaged it for debugging. What do you think? If e2e can have 
yarn's ability, I can try adding e2e
   
   thanks @LiJie20190102 We need to modify these core contents and add e2e, 
which will facilitate future maintenance. What do you think? Currently, the 
default is to use `--master local --deploy-mode client`, and you can refer to 
`AbstractTestSparkContainer` for implementation. Can you help us implement it?


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