Eights-Li commented on pull request #4036: URL: https://github.com/apache/incubator-dolphinscheduler/pull/4036#issuecomment-728645713
> Overall, there seems to be no problem. > But there are two suggestions: > 1: In the parameter construction, I think there is no need to use linkedList, the original StringBuilder may be better. > 2: It is recommended not to use values in the enum. The recommended approach is to put the enumeration value in a static constant. 1 The purpose of using linkedlist is not to splice a large number of spaces in stringbuilder 2 the enum value in a static constant seems a good idea ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
