Suggestion: Support Property Constraints

[pjeanjean]

Hi,

This issue is a follow-up to a discussion on the Discord server.

Currently, ArcadeDB doesn't have support to define constraints on properties other than their type.
The only property setting currently implemented is default, which works nice but isn't properly documented in the manual from what I can tell.

Now that the project is pretty mature, users would benefit a lot if the constraints defined here were implemented:

arcadedb/engine/src/main/java/com/arcadedb/query/sql/parser/CreatePropertyAttributeStatement.java

Lines 79 to 107 in 351583f

	// String attrName = settingName.getStringValue();
	// Object attrValue = this.settingValue == null ? true : this.settingValue.execute((Identifiable) null, ctx);
	// try {
	// if (attrName.equalsIgnoreCase("readonly")) {
	// internalProp.setReadonly((boolean) attrValue);
	// } else if (attrName.equalsIgnoreCase("mandatory")) {
	// internalProp.setMandatory((boolean) attrValue);
	// } else if (attrName.equalsIgnoreCase("notnull")) {
	// internalProp.setNotNull((boolean) attrValue);
	// } else if (attrName.equalsIgnoreCase("max")) {
	// internalProp.setMax("" + attrValue);
	// } else if (attrName.equalsIgnoreCase("min")) {
	// internalProp.setMin("" + attrValue);
	// } else if (attrName.equalsIgnoreCase("default")) {
	// if(this.settingValue==null){
	// throw new PCommandExecutionException("");
	// }
	// internalProp.setDefaultValue("" + attrValue);
	// } else if (attrName.equalsIgnoreCase("collate")) {
	// internalProp.setCollate("" + attrValue);
	// } else if (attrName.equalsIgnoreCase("regexp")) {
	// internalProp.setRegexp("" + attrName);
	// } else {
	// throw new PCommandExecutionException("Invalid attribute definition: '" + attrName + "'");
	// }
	// } catch (Exception e) {
	// throw new PCommandExecutionException("Cannot set attribute on property " + settingName.getStringValue() + " " + attrValue, e);
	// }
	// return attrValue;

This would also bring the project closer to feature parity with OrientDB.

[lvca]

Nice, I was able to reuse >90% of the code + tests from OrientDB. Still finishing the last things and it should be released soon.

[lvca]

The following are going to be supported:

• mandatory (the property must be present)
• readonly (the property cannot be changed after the creation of the record)
• min (minimum value)
• max (maximum value)
• regexp (Regular Expression)