You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Maxim Muzafarov (Jira)" <ji...@apache.org> on 2022/12/04 19:24:00 UTC

[jira] [Created] (CASSANDRA-18089) The source code must obey the avoid star import checkstyle rule

Maxim Muzafarov created CASSANDRA-18089:
-------------------------------------------

             Summary: The source code must obey the avoid star import checkstyle rule
                 Key: CASSANDRA-18089
                 URL: https://issues.apache.org/jira/browse/CASSANDRA-18089
             Project: Cassandra
          Issue Type: Improvement
            Reporter: Maxim Muzafarov
            Assignee: Maxim Muzafarov


Cassandra has the code style rules regarding the classes import order: [https://cassandra.apache.org/_/development/code_style.html]

Importing all classes from a package or static members from a class leads to tight coupling between packages or classes and might lead to problems when a new library version introduces name clashes. The advantage of explicitly listing all imports from a package is that you can tell at a glance which class you meant to use, which does reading and refactoring the source code that much easier.


The checkstyle that is already used for checking the source code has a such check and this check may be added to the config both for the production and test source code:
https://checkstyle.sourceforge.io/config_imports.html#AvoidStaticImport

Besides adding a new checkstyle rule it may be more convenient for those community members that are working with the code to reflect the same rule in the IDE's inspection profiles (if it's possible), thus using the 'optimize imports' will produce the same results on each execution as the checkstyle does.


Summary:
- add new checkstyle rule;
- update IDE's appropriate built-in inspections configurations;
- update development code-style web page and wiki;





--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cassandra.apache.org
For additional commands, e-mail: commits-help@cassandra.apache.org