Xuxiaotuan opened a new pull request, #3571:
URL: https://github.com/apache/incubator-streampark/pull/3571
1. FOR`collection`returned values, unless there are special`concurrent`(such
as thread safety), always return the`interface`, such as:
returns List if use ArrayList
3.Use isEmpty() instead of length() == 0 or size() == 0
I didn't find this way of writing in the `streampark-flink` module.
## What changes were proposed in this pull request
Issue Number: close #3085
<!--(For example: This pull request proposed to add checkstyle plugin).-->
## Brief change log
<!--*(for example:)*
- *Add maven-checkstyle-plugin to root pom.xml*
-->
## Verifying this change
<!--*(Please pick either of the following options)*-->
This change is a trivial rework / code cleanup without any test coverage.
## Does this pull request potentially affect one of the following parts
- Dependencies (does it add or upgrade a dependency): no
--
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]
