[IGNITE-11836] keyFields must be empty if keyType is an SQL type

• Type: Improvement
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Component/s: sql
• Ignite Flags: Docs Required
• Assignee: Unassigned
• Reporter: Stanislav Lukyanov
• Created: 07/May/19 00:47
• Updated: 07/May/19 00:47

Description

Currently if QueryEntity.keyType is an SQL type (e.g. String or Integer) then keyFields property is simply ignored. This creates room for a misconfiguration.

Let's add a check that keyFields is null or empty if keyType is an SQL type.

Probably the same should be done for valueType and valueFields.

Also, fields property should be handled somehow. An obvious case is that fields should be null or empty when both key and value are SQL types. There is probably more invariants to enforce though.