[NIFI-716] Framework allows you to define two properties with the same name

• Type: Bug
• Status: Open
• Priority: Minor
• Resolution: Unresolved
• Affects Version/s: 0.1.0
• Component/s: Core Framework
• Labels: beginner newbie
• Assignee: Unassigned
• Reporter: Dan Bress
• Created: 23/Jun/15 12:31
• Updated: 23/Jun/15 15:16

Description

If you are lazy and copy and paste a PropertyDescriptor and forget to change the value assigned to name(), the framework(test or regular) does not detect this and proceeds happily.

It would be great if this situation was detected as soon as possible, and either mark the processor as invalid, or fail to consider it as a possible processor.

This applies to Processors, ControllerServices and ReportingTasks

Example

public static final PropertyDescriptor MIN_SIZE = new PropertyDescriptor.Builder()
.name("Minimum Group Size")
.description("The minimum size of for the bundle")
.required(true)
.defaultValue("0 B")
.addValidator(StandardValidators.DATA_SIZE_VALIDATOR)
.build();

public static final PropertyDescriptor MAX_SIZE = new PropertyDescriptor.Builder()
.name("Minimum Group Size")
.description("The maximum size for the bundle. If not specified, there is no maximum.")
.required(false)
.addValidator(StandardValidators.DATA_SIZE_VALIDATOR)
.build();

Issue Links

• Is contained by: NIFI-34 Mock Framework should provide option to detect common bad practices/bugs