jerqi commented on PR #935: URL: https://github.com/apache/incubator-uniffle/pull/935#issuecomment-1583808937
> > > > @lifeSo It's caused by your pr. Your file don't include Apache license header. > > > > > > > > > ok, I understand, I re-commit file, please check again. > > > > > > @lifeSo Could you add ut for this class? > > Tez code 耦合性 is too big and test case is not easy to write... You can refer to MR test code. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
