Github user twalthr commented on the issue: https://github.com/apache/flink/pull/3943 @wuchong @sunjincheng121 @fhueske I tried to simplify the package structure a little bit. I based my changes on top of this PRs commit. My goals: - splitting a class like (`ExpressionReductionTest`) in 6 classes is not very helpful - tests are named like the operator or feature that they are testing e.g. `Calc`, `Distinct` etc. (we don't need `CastingITCase`, `CalcITCase`, `ScalarFunctionITCase`; `CalcITCase` covers everything) - all tests have the same name everywhere (`OverWindowTest`, `OverWindowValidationTest`, `OverWindowITCase`, `OverWindowStringExpressionTest`) - no packages with one class - `org.apache.flink.table.api` contains all tests that test the translation from batch/stream/table/sql APIs - `org.apache.flink.table.plan` contains all tests that modify the plan - `org.apache.flink.table.runtime` contains all ITCases and runtime relevant tests - all other package contain unit tests for the classes of the same package You can find my branch here: https://github.com/apache/flink/compare/master...twalthr:FLINK-6617
--- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---