[GitHub] [dolphinscheduler] zhongjiajie commented on pull request #14606: [Fix-14566][master] Fix serial_wait command_params does not replace global_params

["zhongjiajie (via GitHub)" <gi...@apache.org>]

zhongjiajie commented on PR #14606:
URL: https://github.com/apache/dolphinscheduler/pull/14606#issuecomment-1648895697

   > So far I've no idea why `SqlServerDataSource` or `HiveDataSource` e2e-test failed due to timeout error.
   > 
   > I think may need to try upgrading version `test-containers-java` to see if it can be fixed. But this will take some time. In the mean time, Can we temporarily not perform these two tests? @EricGao888 @zhongjiajie @ruanwenjun
   
   <img width="1341" alt="image" src="https://github.com/apache/dolphinscheduler/assets/15820530/93c35065-f375-4c44-9b7f-31131fc45a76">
   
   How about we make those tests optional for pr merged by add a new job for it? we still run but not required ci


-- 
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: commits-unsubscribe@dolphinscheduler.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org