Azure / Azure/data-api-builder

Validate `linking.source.fields` and `linking.source.fields` define valid backing columns of `linking.object`

Open
#2,144 2 comments 0 reactions 1 assignee Claimed by @aaronburtle View on GitHub
needs discussion validation
Dominant language
C#
Stars
1.5k
Forks
370
Avg merge
3d 17h
Merged PRs (30d)
8

Description

Relates to https://github.com/Azure/data-api-builder/issues/1935

Validate that the fields listed in the below config properties are valid backing columns.
- `linking-source-fields`
- `linking-target-fields`

https://learn.microsoft.com/azure/data-api-builder/reference-configuration#many-to-many

When we do our validation of the config in the `RuntimeConfigValidator` one of the things that we would like to validate is that all of the source and target fields in an entity's relationships are valid backing columns in the backend database to which they ought to belong. This is easy enough for non-linking fields, as they exist in the source and target entities. Since source and target entities are defined in our config file, we have data structures which hold the relevant information for easy retrieval during validation.

However, for linking objects this is not necessarily the case. Linking objects do not currently have to be defined as entities in our config, and therefore, to validate that the linking source and linking target fields that reference such a linking object are in fact valid columns within that linking object would require to go back to the backend and query the system tables. This is an expensive call and outside the scope of the `RuntimeConfigValidator` class. Therefore, we should pre-load this information into our data model so that we can quickly retrieve it at time of validation.

Once that is possible we can add the validation step to the validator.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.