Tim Satke

Hi, please move to xqueries/xdb and find that issue. We no longer work in this repo xqueries/xdb#1

@tomarrell is inactive since about febuary, and I'm not an admin in this repo, so I can't archive it or anything

the goal set of the new project is just a different roadmap really

Let's take this discussion to xqueries/xdb#1

Yes, I would fully agree with that description! Feel free to choose either one, you're welcome in both projects!

I like it! Looks good so far

Also important, but optional for this PR, specify the protobuf messages that need to be used. @SUMUKHA-PK you can do this in here if you like, otherwise, we can get...

For a builtin function to be considered "implemented", there needs to be an e2e test in `internal/test` (in addition to required unit tests). See #187 for an example of an...

@SUMUKHA-PK the hook is there in the server, @Abby3017 please figure out (with @SUMUKHA-PK 's help) how to refactor this to use the hook.

Most important feature right now: `ast.Validate(*ast.SQLStmt) error` All other methods should be exported too and should have the form `ast.ValidateXXX(*ast.XXX) error`.