Results 134 comments of kezhenxu94

Thank you all @zirain @ericvn @kfaseela @linsun for your reviews, I have update the PR to address your comments. Let me know if anything else is needed to modify

@ericvn thanks for reviewing, comments are addressed

Many files are missing license headers

> > > BTW, there are two ways to fully migrate those tests from jUnit 4 to jUnit 5: > > > > > > 1. We keep both versions...

> I just worry if we still keep the Junit4, we need to constantly remind contributors to use Junit5. Agree, same concern to me

Powermock is getting more and more annoying such as https://github.com/apache/dolphinscheduler/runs/7929368593?check_suite_focus=true https://github.com/powermock/powermock/issues/1112 I'd rather just review and remove all those tests involving Powermock...

Good to hear we can remove powermock 🎉

> I can finish it, plz assign to me. Thank you! Assigned.

> Is this feature implemented already? If yes, then we might want to close the issue. If not, and if @mikechengwei is not working on this anymore, I can pick...

> @Superskyyy @dashanji Is anyone planning to do this? It is stable for years. @rabajaj0509 is interested in this and may work on this after he finishes the GSoC program