You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In #9650, we added a method for models to infer the primary key based on configured tests and/or constraints. We should serialize this as additional metadata in the manifest.json artifact
Acceptance criteria
output of model.infer_unique_key() is written to the manifest.json artifact for each relevant node
Suggested Tests
We already have unit testing on this method! Additional testing will be required for the artifact itself.
Impact to Other Teams
None
Will backports be required?
I don't think so
Context / Additional details
The primary key inference method on ModelNode requires a list of data_tests that apply to the model, which means data_tests must also be parsed by the point that we are able to populate this field on the ModelNode for serialization.
The text was updated successfully, but these errors were encountered:
Housekeeping
Short description
In #9650, we added a method for models to infer the primary key based on configured tests and/or constraints. We should serialize this as additional metadata in the
manifest.json
artifactAcceptance criteria
output of
model.infer_unique_key()
is written to themanifest.json
artifact for each relevantnode
Suggested Tests
We already have unit testing on this method! Additional testing will be required for the artifact itself.
Impact to Other Teams
None
Will backports be required?
I don't think so
Context / Additional details
The primary key inference method on ModelNode requires a list of
data_tests
that apply to the model, which means data_tests must also be parsed by the point that we are able to populate this field on theModelNode
for serialization.The text was updated successfully, but these errors were encountered: