Junrui Lee

Results 20 comments of Junrui Lee

Thanks @zhuzhurk and @1996fanrui for your reviews, I've updated this pr accordingly, PTAL.

@wanglijie95 @zhuzhurk could you help to review this pr? thanks.

@zhuzhurk Thanks for your review, I've updated this pr accordingly, PTAL.

Thanks @zhuzhurk for reviewing, I've updated this pr accordingly. PTAL.

Thanks @zhuzhurk for the thorough review. I have refactored the BatchJobRecoveryTest and JMFailoverITCase based on your comments. PTAL.

> > Thanks, @venkata91, for your contribution! After reviewing this PR, I'm concerned that it entirely removes limit that source parallelism should lower than source jobVertex's max parallelism. And I...

> > > > Thanks, @venkata91, for your contribution! After reviewing this PR, I'm concerned that it entirely removes limit that source parallelism should lower than source jobVertex's max parallelism....

Sorry for the delayed response due to the National Day holiday in China. Currently, there are three key fields related to parallelism: 1. defaultMaxParallelism: This comes from the config option...

Thanks @zhuzhurk for review, I've updated this pr accordingly. PTAL~

Thanks @zhuzhurk for review, I've updated this pr accordingly. PTAL.