Replace `proof_of_sql_parser::intermediate_ast::ResourceId` with `Vec<sqlparser::ast::Ident>` or `sqlparser::ast::ObjectName` in the `proof-of-sql` crate #352
Right now we already have some code that can convert intermediate AST from proof-of-sql-parser we use to sqlparser AST. Now we need to systematically replace instances of proof-of-sql-parser constructs with their corresponding sqlparser ones.
Changes Required
[ ] Replace proof_of_sql_parser::intermediate_ast::ResourceId with Vec<sqlparser::ast::Ident> or sqlparser::ast::ObjectName in the proof-of-sql crate.
Note that this issue is blocked until #350 is complete.
Background and Motivation
This issue is a subtask in https://github.com/spaceandtimelabs/sxt-proof-of-sql/issues/235. In short since we plan to add more SQL features we plan to switch to the
sqlparsercrate which is a feature-rich,no_std-compatible parser used by DataFusion, which is part of the Arrow ecosystem.Right now we already have some code that can convert intermediate AST from
proof-of-sql-parserwe use to sqlparser AST. Now we need to systematically replace instances ofproof-of-sql-parserconstructs with their corresponding sqlparser ones.Changes Required
proof_of_sql_parser::intermediate_ast::ResourceIdwithVec<sqlparser::ast::Ident>orsqlparser::ast::ObjectNamein theproof-of-sqlcrate.Note that this issue is blocked until #350 is complete.