+
Skip to content

Conversation

baitsguy
Copy link
Contributor

@baitsguy baitsguy commented Nov 5, 2024

Simple input/output type validation on the LogicalPlan layer. Note: this doesn't work on the execution layer (SycamoreExecutor) which can have it's own set of validations that are more dynamic and implementation specific.

Next steps:

  1. add schema validations (e.g. nodes that read fields, can assert that the field exists in the schema or has been added to the data in the query)
  2. llm planner correction, get the llm to fix it's plan

@baitsguy baitsguy marked this pull request as ready for review November 5, 2024 12:04
@baitsguy baitsguy requested a review from mdwelsh November 5, 2024 12:04
baitsguy and others added 2 commits November 7, 2024 20:38
@baitsguy baitsguy enabled auto-merge (squash) November 8, 2024 04:45
@baitsguy baitsguy merged commit 6ee75a8 into main Nov 8, 2024
12 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载