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
Currently, the manifest in Rust side is a simple DTO. We should add more checking rules when deserialization. For example, a model can accept refSql, tableReference or baseObject. However, we didn't check it in Rust side.
Description
Currently, the manifest in Rust side is a simple DTO. We should add more checking rules when deserialization. For example, a model can accept
refSql
,tableReference
orbaseObject
. However, we didn't check it in Rust side.Basically, we should follow the original behavior in java side.
https://github.com/Canner/wren-engine/tree/main/wren-base/src/main/java/io/wren/base/dto
Maybe we can also validate the generated JSON by
https://github.com/Canner/WrenAI/blob/main/wren-mdl/mdl.schema.json
Collect the related issues
properties
in the MDL #719The text was updated successfully, but these errors were encountered: