You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@seatunnel.apache.org by GitBox <gi...@apache.org> on 2022/10/11 12:54:59 UTC

[GitHub] [incubator-seatunnel] TyrantLucifer commented on pull request #3065: [Improve] Using Collections.emptyMap() replaced Collections.EMPTY_MAP

TyrantLucifer commented on PR #3065:
URL: https://github.com/apache/incubator-seatunnel/pull/3065#issuecomment-1274645027

   Hi [FlechazoW](https://github.com/FlechazoW), overall LGTM except few nits. The name of your pull request does not include the name of the module involved. A clear name for the pull request would help make the review easier. You can refer to [Coding Guide](https://seatunnel.apache.org/docs/contribution/coding-guide) get more details about how to submit a high quality pull request.


-- 
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: commits-unsubscribe@seatunnel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org