Eric Gao comments

Results 200 comments of


                                            Eric Gao

[Improvement-#9680][task-plugin] #9680 add hive sql log listener, print MapReduce progress log and obtain application_id allows developers to better track tasks

@SbloodyS @zhongjiajie PTAL when available, thanks : )

[Feature-11488][Datax]Datax can submit to YARN

@duhanmin Hello, thanks for submitting this PR and it's a good feature. I've added some comments and please take a look.

[Improvement] [Alert] mask the password of email(#11913)

The commit history of this PR looks pretty wired, could u plz fix it? Thx ![image](https://user-images.githubusercontent.com/34905992/195036020-827b4d62-5f4c-47ac-872b-56e0cf58cc7c.png)

[Feature-10498] Mask the password in the log of sqoop task

related: #10498

[Feature-10498] Mask the password in the log of sqoop task

This is a good feature. Could you please add a UT for it? Thanks.

[Improvement] [Doc] Document content update

Hi @MonsterChenzhuo May I ask whether you are still working on it?

[Improvement-#11608][task-plugin] New time variables are added to facilitate business development

@ruanwenjun @caishunfeng @SbloodyS Could u plz help take a look when available? Thanks : )

[Improvement-#11608][task-plugin] New time variables are added to facilitate business development

> > I've restarted failed jobs in CI, LGTM once CI passes. > > What do u want me to do ? Nothing actually, just wait for the CI to...

[Improvement-#11608][task-plugin] New time variables are added to facilitate business development

@fengjian1129 Good job! This is a handy feature, thanks again for the contribution!

[Improvement][Test] Block the usage of `Powermock`

BTW, we need to move these two dependencies from each module to root pom. https://github.com/apache/dolphinscheduler/blob/18ef0a66cb91d9dfe6406f8ade6cfb2cff36492e/dolphinscheduler-common/pom.xml#L63-L74