Canner/wren-engine

Enhance the manifest serialization and deserialization

Opened this issue · 0 comments

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 or baseObject. 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

  • #718
  • #717
  • #719
  • Support refSql ( needs more thought for it )