Skip to content

add disk full protection #22

add disk full protection

add disk full protection #22

Triggered via pull request December 17, 2024 12:16
Status Success
Total duration 1m 59s
Artifacts 1

maven.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 10 warnings
Overridable method called during object construction: GRIDAClientMain.initClient() : grida-client/src/main/java/fr/insalyon/creatis/grida/client/GRIDAClientMain.java#L59
Reports calls to overridable methods on `this` during object initialization. These are invoked on an incompletely constructed object and can be difficult to debug if overridden. This is because the subclass usually assumes that the superclass is completely initialized in all methods. If that is not the case, bugs can appear in the constructor, for instance, some fields that are still null may cause a NullPointerException or be stored somewhere else to blow up later. To avoid this problem, only use methods that are static, private, or final in constructors. Note that those methods also must not call overridable methods transitively to be safe. ConstructorCallsOverridableMethod (Priority: 1, Ruleset: Error Prone) https://docs.pmd-code.org/pmd-doc-7.8.0/pmd_rules_java_errorprone.html#constructorcallsoverridablemethod
An empty method in an abstract class should be abstract instead: grida-common/src/main/java/fr/insalyon/creatis/grida/common/Communication.java#L22
Empty or auto-generated methods in an abstract class should be tagged as abstract. This helps to remove their inapproprate usage by developers who should be implementing their own versions in the concrete subclasses. EmptyMethodInAbstractClassShouldBeAbstract (Priority: 1, Ruleset: Code Style) https://docs.pmd-code.org/pmd-doc-7.8.0/pmd_rules_java_codestyle.html#emptymethodinabstractclassshouldbeabstract
The enum constant name 'Folder' doesn't match '[A-Z][A-Z_0-9]*': grida-common/src/main/java/fr/insalyon/creatis/grida/common/bean/GridData.java#L48
Configurable naming conventions for field declarations. This rule reports variable declarations which do not match the regex that applies to their specific kind ---e.g. constants (static final), enum constant, final field. Each regex can be configured through properties. By default this rule uses the standard Java naming convention (Camel case), and uses the ALL_UPPER convention for constants and enum constants. FieldNamingConventions (Priority: 1, Ruleset: Code Style) https://docs.pmd-code.org/pmd-doc-7.8.0/pmd_rules_java_codestyle.html#fieldnamingconventions
The enum constant name 'File' doesn't match '[A-Z][A-Z_0-9]*': grida-common/src/main/java/fr/insalyon/creatis/grida/common/bean/GridData.java#L48
Configurable naming conventions for field declarations. This rule reports variable declarations which do not match the regex that applies to their specific kind ---e.g. constants (static final), enum constant, final field. Each regex can be configured through properties. By default this rule uses the standard Java naming convention (Camel case), and uses the ALL_UPPER convention for constants and enum constants. FieldNamingConventions (Priority: 1, Ruleset: Code Style) https://docs.pmd-code.org/pmd-doc-7.8.0/pmd_rules_java_codestyle.html#fieldnamingconventions
The enum constant name 'Upload' doesn't match '[A-Z][A-Z_0-9]*': grida-common/src/main/java/fr/insalyon/creatis/grida/common/bean/Operation.java#L49
Configurable naming conventions for field declarations. This rule reports variable declarations which do not match the regex that applies to their specific kind ---e.g. constants (static final), enum constant, final field. Each regex can be configured through properties. By default this rule uses the standard Java naming convention (Camel case), and uses the ALL_UPPER convention for constants and enum constants. FieldNamingConventions (Priority: 1, Ruleset: Code Style) https://docs.pmd-code.org/pmd-doc-7.8.0/pmd_rules_java_codestyle.html#fieldnamingconventions
The enum constant name 'Download' doesn't match '[A-Z][A-Z_0-9]*': grida-common/src/main/java/fr/insalyon/creatis/grida/common/bean/Operation.java#L49
Configurable naming conventions for field declarations. This rule reports variable declarations which do not match the regex that applies to their specific kind ---e.g. constants (static final), enum constant, final field. Each regex can be configured through properties. By default this rule uses the standard Java naming convention (Camel case), and uses the ALL_UPPER convention for constants and enum constants. FieldNamingConventions (Priority: 1, Ruleset: Code Style) https://docs.pmd-code.org/pmd-doc-7.8.0/pmd_rules_java_codestyle.html#fieldnamingconventions
The enum constant name 'Download_Files' doesn't match '[A-Z][A-Z_0-9]*': grida-common/src/main/java/fr/insalyon/creatis/grida/common/bean/Operation.java#L49
Configurable naming conventions for field declarations. This rule reports variable declarations which do not match the regex that applies to their specific kind ---e.g. constants (static final), enum constant, final field. Each regex can be configured through properties. By default this rule uses the standard Java naming convention (Camel case), and uses the ALL_UPPER convention for constants and enum constants. FieldNamingConventions (Priority: 1, Ruleset: Code Style) https://docs.pmd-code.org/pmd-doc-7.8.0/pmd_rules_java_codestyle.html#fieldnamingconventions
The enum constant name 'Delete' doesn't match '[A-Z][A-Z_0-9]*': grida-common/src/main/java/fr/insalyon/creatis/grida/common/bean/Operation.java#L49
Configurable naming conventions for field declarations. This rule reports variable declarations which do not match the regex that applies to their specific kind ---e.g. constants (static final), enum constant, final field. Each regex can be configured through properties. By default this rule uses the standard Java naming convention (Camel case), and uses the ALL_UPPER convention for constants and enum constants. FieldNamingConventions (Priority: 1, Ruleset: Code Style) https://docs.pmd-code.org/pmd-doc-7.8.0/pmd_rules_java_codestyle.html#fieldnamingconventions
The enum constant name 'Replicate' doesn't match '[A-Z][A-Z_0-9]*': grida-common/src/main/java/fr/insalyon/creatis/grida/common/bean/Operation.java#L49
Configurable naming conventions for field declarations. This rule reports variable declarations which do not match the regex that applies to their specific kind ---e.g. constants (static final), enum constant, final field. Each regex can be configured through properties. By default this rule uses the standard Java naming convention (Camel case), and uses the ALL_UPPER convention for constants and enum constants. FieldNamingConventions (Priority: 1, Ruleset: Code Style) https://docs.pmd-code.org/pmd-doc-7.8.0/pmd_rules_java_codestyle.html#fieldnamingconventions
The enum constant name 'Queued' doesn't match '[A-Z][A-Z_0-9]*': grida-common/src/main/java/fr/insalyon/creatis/grida/common/bean/Operation.java#L54
Configurable naming conventions for field declarations. This rule reports variable declarations which do not match the regex that applies to their specific kind ---e.g. constants (static final), enum constant, final field. Each regex can be configured through properties. By default this rule uses the standard Java naming convention (Camel case), and uses the ALL_UPPER convention for constants and enum constants. FieldNamingConventions (Priority: 1, Ruleset: Code Style) https://docs.pmd-code.org/pmd-doc-7.8.0/pmd_rules_java_codestyle.html#fieldnamingconventions
This abstract class does not have any abstract methods: grida-client/src/main/java/fr/insalyon/creatis/grida/client/AbstractGRIDAClient.java#L48
The abstract class does not contain any abstract methods. An abstract class suggests an incomplete implementation, which is to be completed by subclasses implementing the abstract methods. If the class is intended to be used as a base class only (not to be instantiated directly) a protected constructor can be provided to prevent direct instantiation. AbstractClassWithoutAbstractMethod (Priority: 3, Ruleset: Best Practices) https://docs.pmd-code.org/pmd-doc-7.8.0/pmd_rules_java_bestpractices.html#abstractclasswithoutabstractmethod
Field 'host' may be declared final: grida-client/src/main/java/fr/insalyon/creatis/grida/client/AbstractGRIDAClient.java#L50
Reports non-final fields whose value never changes once object initialization ends, and hence may be marked final. Note that this rule does not enforce that the field value be deeply immutable itself. An object can still have mutable state, even if all its member fields are declared final. This is referred to as shallow immutability. For more information on mutability, see *Effective Java, 3rd Edition, Item 17: Minimize mutability*. Limitations: We can only check private fields for now. ImmutableField (Priority: 3, Ruleset: Design) https://docs.pmd-code.org/pmd-doc-7.8.0/pmd_rules_java_design.html#immutablefield
Field 'port' may be declared final: grida-client/src/main/java/fr/insalyon/creatis/grida/client/AbstractGRIDAClient.java#L51
Reports non-final fields whose value never changes once object initialization ends, and hence may be marked final. Note that this rule does not enforce that the field value be deeply immutable itself. An object can still have mutable state, even if all its member fields are declared final. This is referred to as shallow immutability. For more information on mutability, see *Effective Java, 3rd Edition, Item 17: Minimize mutability*. Limitations: We can only check private fields for now. ImmutableField (Priority: 3, Ruleset: Design) https://docs.pmd-code.org/pmd-doc-7.8.0/pmd_rules_java_design.html#immutablefield
Parameter 'host' is not assigned and could be declared final: grida-client/src/main/java/fr/insalyon/creatis/grida/client/AbstractGRIDAClient.java#L54
Reports method and constructor parameters that can be made final because they are never reassigned within the body of the method. This rule ignores unused parameters so as not to overlap with the rule {% rule java/bestpractices/UnusedFormalParameter %}. It will also ignore the parameters of abstract methods. MethodArgumentCouldBeFinal (Priority: 3, Ruleset: Code Style) https://docs.pmd-code.org/pmd-doc-7.8.0/pmd_rules_java_codestyle.html#methodargumentcouldbefinal
Parameter 'port' is not assigned and could be declared final: grida-client/src/main/java/fr/insalyon/creatis/grida/client/AbstractGRIDAClient.java#L54
Reports method and constructor parameters that can be made final because they are never reassigned within the body of the method. This rule ignores unused parameters so as not to overlap with the rule {% rule java/bestpractices/UnusedFormalParameter %}. It will also ignore the parameters of abstract methods. MethodArgumentCouldBeFinal (Priority: 3, Ruleset: Code Style) https://docs.pmd-code.org/pmd-doc-7.8.0/pmd_rules_java_codestyle.html#methodargumentcouldbefinal
Parameter 'proxyPath' is not assigned and could be declared final: grida-client/src/main/java/fr/insalyon/creatis/grida/client/AbstractGRIDAClient.java#L54
Reports method and constructor parameters that can be made final because they are never reassigned within the body of the method. This rule ignores unused parameters so as not to overlap with the rule {% rule java/bestpractices/UnusedFormalParameter %}. It will also ignore the parameters of abstract methods. MethodArgumentCouldBeFinal (Priority: 3, Ruleset: Code Style) https://docs.pmd-code.org/pmd-doc-7.8.0/pmd_rules_java_codestyle.html#methodargumentcouldbefinal
'catch' branch identical to 'UnknownHostException' branch: grida-client/src/main/java/fr/insalyon/creatis/grida/client/AbstractGRIDAClient.java#L67
Identical `catch` branches use up vertical space and increase the complexity of code without adding functionality. It's better style to collapse identical branches into a single multi-catch branch. IdenticalCatchBranches (Priority: 3, Ruleset: Code Style) https://docs.pmd-code.org/pmd-doc-7.8.0/pmd_rules_java_codestyle.html#identicalcatchbranches
Parameter 'host' is not assigned and could be declared final: grida-client/src/main/java/fr/insalyon/creatis/grida/client/GRIDACacheClient.java#L59
Reports method and constructor parameters that can be made final because they are never reassigned within the body of the method. This rule ignores unused parameters so as not to overlap with the rule {% rule java/bestpractices/UnusedFormalParameter %}. It will also ignore the parameters of abstract methods. MethodArgumentCouldBeFinal (Priority: 3, Ruleset: Code Style) https://docs.pmd-code.org/pmd-doc-7.8.0/pmd_rules_java_codestyle.html#methodargumentcouldbefinal
Parameter 'port' is not assigned and could be declared final: grida-client/src/main/java/fr/insalyon/creatis/grida/client/GRIDACacheClient.java#L59
Reports method and constructor parameters that can be made final because they are never reassigned within the body of the method. This rule ignores unused parameters so as not to overlap with the rule {% rule java/bestpractices/UnusedFormalParameter %}. It will also ignore the parameters of abstract methods. MethodArgumentCouldBeFinal (Priority: 3, Ruleset: Code Style) https://docs.pmd-code.org/pmd-doc-7.8.0/pmd_rules_java_codestyle.html#methodargumentcouldbefinal
Parameter 'proxyPath' is not assigned and could be declared final: grida-client/src/main/java/fr/insalyon/creatis/grida/client/GRIDACacheClient.java#L59
Reports method and constructor parameters that can be made final because they are never reassigned within the body of the method. This rule ignores unused parameters so as not to overlap with the rule {% rule java/bestpractices/UnusedFormalParameter %}. It will also ignore the parameters of abstract methods. MethodArgumentCouldBeFinal (Priority: 3, Ruleset: Code Style) https://docs.pmd-code.org/pmd-doc-7.8.0/pmd_rules_java_codestyle.html#methodargumentcouldbefinal

Artifacts

Produced during runtime
Name Size
PMD Report
54.2 KB